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

Allow MockConnector to support table and schema properties #10618

Merged
merged 2 commits into from
Jan 22, 2022

Conversation

kokosing
Copy link
Member

Allow MockConnector to support table and schema properties

@cla-bot cla-bot bot added the cla-signed label Jan 14, 2022
@kokosing kokosing force-pushed the origin/master/297_mock_props branch from 0d0222f to 62cf13b Compare January 14, 2022 15:11
@kokosing kokosing force-pushed the origin/master/297_mock_props branch from 62cf13b to 33df6f5 Compare January 20, 2022 11:47
@kokosing
Copy link
Member Author

@gaurav8297 @Praveen2112 AC, PTAL

Procedures do not configure access control in MockConnector,
so system access control should be used.
@kokosing kokosing force-pushed the origin/master/297_mock_props branch from 33df6f5 to 4453e36 Compare January 21, 2022 11:52
@kokosing kokosing force-pushed the origin/master/297_mock_props branch from 4453e36 to 25bead9 Compare January 21, 2022 14:09
@kokosing kokosing merged commit e6c62cd into trinodb:master Jan 22, 2022
@kokosing kokosing deleted the origin/master/297_mock_props branch January 22, 2022 20:23
@github-actions github-actions bot added this to the 369 milestone Jan 22, 2022
linzebing pushed a commit to linzebing/trino that referenced this pull request Jan 26, 2022
…0618)

* Do not provide access control for procedures in MockConnector

Procedures do not configure access control in MockConnector,
so system access control should be used.

* Allow MockConnector to support table and schema properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants