Skip to content

Adapt test to use DENY statement#12716

Merged
kokosing merged 1 commit intotrinodb:masterfrom
findinpath:deny-test-fix
Jun 7, 2022
Merged

Adapt test to use DENY statement#12716
kokosing merged 1 commit intotrinodb:masterfrom
findinpath:deny-test-fix

Conversation

@findinpath
Copy link
Copy Markdown
Contributor

Description

Adapt the test for DENY statement to test DENY instead of GRANT when being applied on a missing table.

Is this change a fix, improvement, new feature, refactoring, or other?

Test fix

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

N/A

How would you describe this change to a non-technical end user or system administrator?

N/A

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

@cla-bot cla-bot bot added the cla-signed label Jun 7, 2022
@findinpath findinpath requested a review from dain June 7, 2022 05:06
@kokosing kokosing merged commit 56013a0 into trinodb:master Jun 7, 2022
@kokosing
Copy link
Copy Markdown
Member

kokosing commented Jun 7, 2022

Thanks, merged.

@github-actions github-actions bot added this to the 385 milestone Jun 7, 2022
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