Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sql: Add information_schema.constraint_column_usage table #22323

Conversation

nvanbenschoten
Copy link
Member

@nvanbenschoten nvanbenschoten commented Feb 2, 2018

Related to #8675.
Related to #22298.

The constraint_column_usage table identifies all columns in the current
database that are used by some constraint.

Release note (sql change): Added constraint_column_usage table to the
information_schema.

@nvanbenschoten nvanbenschoten requested review from BramGruneir and a team February 2, 2018 10:05
@cockroach-teamcity
Copy link
Member

This change is Reviewable

…aints

Now that we have this information available, we can properly populate
this column.

Release note: None
The `constraint_column_usage` table identifies all columns in the current
database that are used by some constraint.

Release note (sql change): Added constraint_column_usage table to the
information_schema.
@nvanbenschoten nvanbenschoten force-pushed the nvanbenschoten/constraintColumnUsage2 branch from 8201753 to 0bda411 Compare February 7, 2018 21:46
@nvanbenschoten
Copy link
Member Author

Rebased on top of #22322. PTAL @BramGruneir!

@BramGruneir
Copy link
Member

:lgtm:


Reviewed 2 of 2 files at r1, 2 of 2 files at r2, 3 of 3 files at r3.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@nvanbenschoten nvanbenschoten merged commit d1b4fdb into cockroachdb:master Feb 12, 2018
@nvanbenschoten
Copy link
Member Author

TFTR @BramGruneir!

cc. @knz as this will affect the rebase of #22371.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants