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

tableacls: remove deprecated -legacy-table-acl option from vttablet #4756

Merged
merged 1 commit into from
Mar 31, 2019

Conversation

dweitzman
Copy link
Member

If anyone out there is using legacy behavior, seems like it should
be possible to switch to the newer behavior.

The legacy behavior has a weaker understanding of what queries do
and seems like if anything would be stricter than the newer behavior.

@dweitzman dweitzman requested a review from sougou as a code owner March 28, 2019 01:12
If anyone out there is using legacy behavior, seems like it should
be possible to switch to the newer behavior.

The legacy behavior has a weaker understanding of what queries do
and seems like if anything would be stricter than the newer behavior.

Signed-off-by: David Weitzman <dweitzman@pinterest.com>
@sougou sougou merged commit 8d0c1a9 into vitessio:master Mar 31, 2019
@dweitzman dweitzman deleted the legacy_table_acl branch June 18, 2019 17:40
notfelineit pushed a commit to planetscale/vitess that referenced this pull request Apr 13, 2024
Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>
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.

2 participants