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

Support all privileges of MariaDB 10.6 in MariaDB grammar #3075

Merged
merged 1 commit into from Feb 13, 2023

Conversation

joschi
Copy link
Contributor

@joschi joschi commented Feb 13, 2023

Some privileges supported by MariaDB are missing from the grammar.

This change set adds the missing tokens and tests them against a GRANT query with all privileges supported by MariaDB 10.6.x: https://mariadb.com/docs/server/ref/es10.6/privileges/

@KvanTTT KvanTTT self-assigned this Feb 13, 2023
Copy link
Member

@KvanTTT KvanTTT left a comment

Choose a reason for hiding this comment

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

We already have separated grammar for MariaDB: https://github.com/antlr/grammars-v4/tree/master/sql/mariadb

@uurl could you check if it's correct to add MariaDB changes to the current grammar? Or they should be added to the separated grammar.

sql/mysql/Positive-Technologies/MySqlLexer.g4 Outdated Show resolved Hide resolved
@KvanTTT KvanTTT added example New example of file(s) parsed by grammar-generated parser mariadb labels Feb 13, 2023
@joschi
Copy link
Contributor Author

joschi commented Feb 13, 2023

We already have separated grammar for MariaDB: https://github.com/antlr/grammars-v4/tree/master/sql/mariadb

@KvanTTT Yes, I saw that and updated this MR. 😄

@joschi joschi changed the title Support all privileges of MariaDB 10.6 in MySQL grammar Support all privileges of MariaDB 10.6 in MariaDB grammar Feb 13, 2023
@joschi joschi requested a review from KvanTTT February 13, 2023 16:20
@KvanTTT KvanTTT merged commit 67f35c9 into antlr:master Feb 13, 2023
@joschi joschi deleted the mariadb-privileges branch February 13, 2023 17:05
joschi added a commit to joschi/debezium that referenced this pull request Apr 5, 2023
joschi added a commit to joschi/debezium that referenced this pull request Apr 5, 2023
jpechane pushed a commit to joschi/debezium that referenced this pull request Apr 13, 2023
jpechane pushed a commit to debezium/debezium that referenced this pull request Apr 13, 2023
roldanbob pushed a commit to roldanbob/debezium that referenced this pull request Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
example New example of file(s) parsed by grammar-generated parser mariadb
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants