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: populate information_schema.table_privileges.with_hierarchy #35578

Merged
merged 1 commit into from
Mar 9, 2019

Conversation

knz
Copy link
Contributor

@knz knz commented Mar 9, 2019

Pg doc says:

In the SQL standard, WITH HIERARCHY OPTION is a
separate (sub-)privilege allowing certain operations on table
inheritance hierarchies. In PostgreSQL, this is included in the SELECT
privilege, so this column shows YES if the privilege is SELECT, else
NO.

https://www.postgresql.org/docs/9.5/infoschema-table-privileges.html

Release note: None

Pg doc says:

*In the SQL standard, WITH HIERARCHY OPTION is a
separate (sub-)privilege allowing certain operations on table
inheritance hierarchies. In PostgreSQL, this is included in the SELECT
privilege, so this column shows YES if the privilege is SELECT, else
NO.*

https://www.postgresql.org/docs/9.5/infoschema-table-privileges.html

Release note: None
@knz knz requested review from a team March 9, 2019 16:04
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Member

@BramGruneir BramGruneir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @bobvawter and @mjibson)

@knz
Copy link
Contributor Author

knz commented Mar 9, 2019

TFYR!

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 9, 2019

Build failed (retrying...)

@craig
Copy link
Contributor

craig bot commented Mar 9, 2019

Build failed (retrying...)

@craig
Copy link
Contributor

craig bot commented Mar 9, 2019

Build failed

@knz
Copy link
Contributor Author

knz commented Mar 9, 2019

requeue after #35583

@knz
Copy link
Contributor Author

knz commented Mar 9, 2019

requeue after #35583 merges

@knz
Copy link
Contributor Author

knz commented Mar 9, 2019

bors r+

craig bot pushed a commit that referenced this pull request Mar 9, 2019
35573: sql/parser: add a missing contextual help annotation r=knz a=knz

Release note: None

35575: sql: fix the match type reporting in information_schema.referential_constraints r=knz a=knz

This is the info_schema complement to #35052.

Release note (bug fix): CockroachDB now properly reports the composite
foreign key match type in `information_schema.referential_constraints`.

35577: sql: properly mark sequences as `bigint` in information_schema.sequences r=knz a=knz

Release note (bug fix): CockroachDB now properly reports `bigint`
in information_schema.sequences.type, for compatibility with PostgreSQL.

35578: sql: populate information_schema.table_privileges.with_hierarchy r=knz a=knz

Pg doc says:

*In the SQL standard, WITH HIERARCHY OPTION is a
separate (sub-)privilege allowing certain operations on table
inheritance hierarchies. In PostgreSQL, this is included in the SELECT
privilege, so this column shows YES if the privilege is SELECT, else
NO.*

https://www.postgresql.org/docs/9.5/infoschema-table-privileges.html

Release note: None

Co-authored-by: Raphael 'kena' Poss <[email protected]>
@craig
Copy link
Contributor

craig bot commented Mar 9, 2019

Build succeeded

@craig craig bot merged commit 04d8331 into cockroachdb:master Mar 9, 2019
@knz knz deleted the 20190309-with-hierarchy branch March 9, 2019 22:12
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