This is an automated email from the ASF dual-hosted git repository. panjuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/shardingsphere.git
The following commit(s) were added to refs/heads/master by this push: new ba1d71c add show create user (#12891) ba1d71c is described below commit ba1d71caf433d4e4bc2d27c3c586a949e7b34bef Author: strawberry-crisis <qiaod...@hotmail.com> AuthorDate: Tue Oct 5 18:35:23 2021 +0800 add show create user (#12891) * add show create user * add show create user * fix integration test --- .../impl/MySQLDALStatementSQLVisitor.java | 11 +++++- .../mysql/dal/MySQLShowCreateUserStatement.java | 36 +++++++++++++++++ .../asserts/statement/dal/DALStatementAssert.java | 5 +++ .../dal/impl/ShowCreateUserStatementAssert.java | 46 ++++++++++++++++++++++ .../jaxb/cases/domain/SQLParserTestCases.java | 5 +++ .../domain/segment/impl/user/ExpectedUser.java | 30 ++++++++++++++ .../dal/ShowCreateUserStatementTestCase.java | 36 +++++++++++++++++ .../src/main/resources/case/dal/show.xml | 4 ++ .../src/main/resources/sql/supported/dal/show.xml | 1 + 9 files changed, 173 insertions(+), 1 deletion(-) diff --git a/shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/statement/impl/MySQLDALStatementSQLVisitor.java b/shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/statement/impl/MySQLDALStatementSQLVisitor.java index f979a19..5249d6c 100644 --- a/shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/statement/impl/MySQLDALStatementSQLVisitor.java +++ b/shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/mysql/visitor/statement/impl/MySQLDALStatementSQLVisitor.java @@ -62,6 +62,7 @@ import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowErr import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowIndexContext; import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowLikeContext; import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowProcesslistContext; +import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowCreateUserContext; import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowStatusContext; import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowTableStatusContext; import org.apache.shardingsphere.sql.parser.autogen.MySQLStatementParser.ShowTablesContext; @@ -108,6 +109,7 @@ import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQ import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateProcedureStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateTableStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateTriggerStatement; +import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateUserStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateViewStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowDatabasesStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowErrorsStatement; @@ -403,7 +405,14 @@ public final class MySQLDALStatementSQLVisitor extends MySQLStatementSQLVisitor public ASTNode visitShowProcesslist(final ShowProcesslistContext ctx) { return new MySQLShowProcessListStatement(); } - + + @Override + public ASTNode visitShowCreateUser(final ShowCreateUserContext ctx) { + MySQLShowCreateUserStatement result = new MySQLShowCreateUserStatement(); + result.setName(((IdentifierValue) visit(ctx.userName())).getValue()); + return result; + } + @Override public ASTNode visitSetVariable(final SetVariableContext ctx) { MySQLSetStatement result = new MySQLSetStatement(); diff --git a/shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/dialect/statement/mysql/dal/MySQLShowCreateUserStatement.java b/shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/dialect/statement/mysql/dal/MySQLShowCreateUserStatement.java new file mode 100644 index 0000000..e4d17e1 --- /dev/null +++ b/shardingsphere-sql-parser/shardingsphere-sql-parser-statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/dialect/statement/mysql/dal/MySQLShowCreateUserStatement.java @@ -0,0 +1,36 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal; + +import lombok.Getter; +import lombok.Setter; +import lombok.ToString; +import org.apache.shardingsphere.sql.parser.sql.common.statement.AbstractSQLStatement; +import org.apache.shardingsphere.sql.parser.sql.common.statement.dal.DALStatement; +import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.MySQLStatement; + +/** + * MySQL show create user statement. + */ +@Getter +@Setter +@ToString +public final class MySQLShowCreateUserStatement extends AbstractSQLStatement implements DALStatement, MySQLStatement { + + private String name; +} diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/DALStatementAssert.java b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/DALStatementAssert.java index d357a26..9bc65be 100644 --- a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/DALStatementAssert.java +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/DALStatementAssert.java @@ -25,6 +25,7 @@ import org.apache.shardingsphere.sql.parser.sql.common.statement.dal.SetStatemen import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowColumnsStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateTableStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateTriggerStatement; +import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateUserStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowDatabasesStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowIndexStatement; import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowTableStatusStatement; @@ -38,6 +39,7 @@ import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowColumnsStatementAssert; import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowCreateTableStatementAssert; import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowCreateTriggerStatementAssert; +import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowCreateUserStatementAssert; import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowDatabasesStatementAssert; import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowIndexStatementAssert; import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl.ShowStatementAssert; @@ -49,6 +51,7 @@ import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowColumnsStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateTableStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateTriggerStatementTestCase; +import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateUserStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowDatabasesStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowIndexStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowStatementTestCase; @@ -85,6 +88,8 @@ public final class DALStatementAssert { ShowCreateTableStatementAssert.assertIs(assertContext, (MySQLShowCreateTableStatement) actual, (ShowCreateTableStatementTestCase) expected); } else if (actual instanceof MySQLShowCreateTriggerStatement) { ShowCreateTriggerStatementAssert.assertIs(assertContext, (MySQLShowCreateTriggerStatement) actual, (ShowCreateTriggerStatementTestCase) expected); + } else if (actual instanceof MySQLShowCreateUserStatement) { + ShowCreateUserStatementAssert.assertIs(assertContext, (MySQLShowCreateUserStatement) actual, (ShowCreateUserStatementTestCase) expected); } else if (actual instanceof MySQLShowTableStatusStatement) { ShowTableStatusStatementAssert.assertIs(assertContext, (MySQLShowTableStatusStatement) actual, (ShowTableStatusStatementTestCase) expected); } else if (actual instanceof MySQLShowIndexStatement) { diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/impl/ShowCreateUserStatementAssert.java b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/impl/ShowCreateUserStatementAssert.java new file mode 100644 index 0000000..f5b5760 --- /dev/null +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/asserts/statement/dal/impl/ShowCreateUserStatementAssert.java @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.apache.shardingsphere.test.sql.parser.parameterized.asserts.statement.dal.impl; + +import lombok.AccessLevel; +import lombok.NoArgsConstructor; +import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.dal.MySQLShowCreateUserStatement; +import org.apache.shardingsphere.test.sql.parser.parameterized.asserts.SQLCaseAssertContext; +import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateUserStatementTestCase; +import org.junit.Assert; + +import static org.hamcrest.CoreMatchers.is; + +/** + * Show create user statement assert. + */ +@NoArgsConstructor(access = AccessLevel.PRIVATE) +public final class ShowCreateUserStatementAssert { + + /** + * Assert show create user statement is correct with expected parser result. + * + * @param assertContext assert context + * @param actual actual show create user statement + * @param expected expected show create user statement test case + */ + public static void assertIs(final SQLCaseAssertContext assertContext, final MySQLShowCreateUserStatement actual, final ShowCreateUserStatementTestCase expected) { + Assert.assertNotNull("expected show create user should be not null", expected.getUser()); + Assert.assertThat(actual.getName(), is(expected.getUser().getName())); + } +} diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/SQLParserTestCases.java b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/SQLParserTestCases.java index d06e8a4..bbad196 100644 --- a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/SQLParserTestCases.java +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/SQLParserTestCases.java @@ -26,6 +26,7 @@ import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowColumnsStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateTableStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateTriggerStatementTestCase; +import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowCreateUserStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowDatabasesStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowIndexStatementTestCase; import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal.ShowStatementTestCase; @@ -293,6 +294,9 @@ public final class SQLParserTestCases { @XmlElement(name = "show-create-trigger") private final List<ShowCreateTriggerStatementTestCase> showCreateTriggerTestCases = new LinkedList<>(); + + @XmlElement(name = "show-create-user") + private final List<ShowCreateUserStatementTestCase> showCreateUserTestCases = new LinkedList<>(); @XmlElement(name = "show-table-status") private final List<ShowTableStatusStatementTestCase> showTableStatusTestCases = new LinkedList<>(); @@ -624,6 +628,7 @@ public final class SQLParserTestCases { putAll(showColumnsTestCases, result); putAll(showCreateTableTestCases, result); putAll(showCreateTriggerTestCases, result); + putAll(showCreateUserTestCases, result); putAll(showTableStatusTestCases, result); putAll(showIndexTestCases, result); putAll(showTestCases, result); diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/segment/impl/user/ExpectedUser.java b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/segment/impl/user/ExpectedUser.java new file mode 100644 index 0000000..b01352f --- /dev/null +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/segment/impl/user/ExpectedUser.java @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.segment.impl.user; + +import lombok.Getter; +import lombok.Setter; +import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.segment.AbstractExpectedIdentifierSQLSegment; + +/** + * Expected user. + */ +@Getter +@Setter +public final class ExpectedUser extends AbstractExpectedIdentifierSQLSegment { +} diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/statement/dal/ShowCreateUserStatementTestCase.java b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/statement/dal/ShowCreateUserStatementTestCase.java new file mode 100644 index 0000000..bbedc90 --- /dev/null +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/java/org/apache/shardingsphere/test/sql/parser/parameterized/jaxb/cases/domain/statement/dal/ShowCreateUserStatementTestCase.java @@ -0,0 +1,36 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. + * The ASF licenses this file to You under the Apache License, Version 2.0 + * (the "License"); you may not use this file except in compliance with + * the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.dal; + +import lombok.Getter; +import lombok.Setter; +import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.segment.impl.user.ExpectedUser; +import org.apache.shardingsphere.test.sql.parser.parameterized.jaxb.cases.domain.statement.SQLParserTestCase; + +import javax.xml.bind.annotation.XmlElement; + +/** + * Show create user statement test case. + */ +@Getter +@Setter +public final class ShowCreateUserStatementTestCase extends SQLParserTestCase { + + @XmlElement + private ExpectedUser user; +} diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/resources/case/dal/show.xml b/shardingsphere-test/shardingsphere-parser-test/src/main/resources/case/dal/show.xml index eb5177f..1e41397 100644 --- a/shardingsphere-test/shardingsphere-parser-test/src/main/resources/case/dal/show.xml +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/resources/case/dal/show.xml @@ -59,6 +59,10 @@ <show-create-trigger sql-case-id="show_create_trigger"> <trigger name="trigger1" start-delimiter="`" end-delimiter="`" start-index="21" stop-index="30" /> </show-create-trigger> + + <show-create-user sql-case-id="show_create_user"> + <user name="user1" start-delimiter="`" end-delimiter="`" start-index="17" stop-index="24" /> + </show-create-user> <show-table-status sql-case-id="show_table_status" /> <show sql-case-id="show_all" /> diff --git a/shardingsphere-test/shardingsphere-parser-test/src/main/resources/sql/supported/dal/show.xml b/shardingsphere-test/shardingsphere-parser-test/src/main/resources/sql/supported/dal/show.xml index 18a025f..51cbe65 100644 --- a/shardingsphere-test/shardingsphere-parser-test/src/main/resources/sql/supported/dal/show.xml +++ b/shardingsphere-test/shardingsphere-parser-test/src/main/resources/sql/supported/dal/show.xml @@ -27,6 +27,7 @@ <sql-case id="show_columns_from_table" value="SHOW COLUMNS FROM `t_order`" db-types="MySQL" /> <sql-case id="show_create_table" value="SHOW CREATE TABLE `t_order`" db-types="MySQL" /> <sql-case id="show_create_trigger" value="SHOW CREATE TRIGGER `trigger1`" db-types="MySQL" /> + <sql-case id="show_create_user" value="SHOW CREATE USER `user1`" db-types="MySQL" /> <sql-case id="show_all" value="SHOW ALL" db-types="PostgreSQL" /> <sql-case id="show_server_version" value="SHOW SERVER_VERSION" db-types="PostgreSQL" /> <sql-case id="show_transaction_isolation_level" value="SHOW TRANSACTION ISOLATION LEVEL" db-types="PostgreSQL" />