You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by zh...@apache.org on 2025/06/27 09:39:20 UTC
(shardingsphere) branch master updated: Rename and move OracleAlterSessionStatement and OracleAlterSystemStatement (#35816)
This is an automated email from the ASF dual-hosted git repository.
zhangliang 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 78402235172 Rename and move OracleAlterSessionStatement and OracleAlterSystemStatement (#35816)
78402235172 is described below
commit 78402235172d77a7861158817c03a8901c6f6f28
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Fri Jun 27 17:39:14 2025 +0800
Rename and move OracleAlterSessionStatement and OracleAlterSystemStatement (#35816)
---
.../visitor/statement/type/OracleDDLStatementVisitor.java | 8 ++++----
.../statement/oracle/ddl/OracleAlterSessionStatement.java} | 8 +++++---
.../statement/oracle/ddl/OracleAlterSystemStatement.java} | 8 +++++---
.../internal/asserts/statement/ddl/DDLStatementAssert.java | 12 ++++++------
.../statement/ddl/impl/AlterSessionStatementAssert.java | 4 ++--
.../statement/ddl/impl/AlterSystemStatementAssert.java | 4 ++--
6 files changed, 24 insertions(+), 20 deletions(-)
diff --git a/parser/sql/dialect/oracle/src/main/java/org/apache/shardingsphere/sql/parser/oracle/visitor/statement/type/OracleDDLStatementVisitor.java b/parser/sql/dialect/oracle/src/main/java/org/apache/shardingsphere/sql/parser/oracle/visitor/statement/type/OracleDDLStatementVisitor.java
index f66a425bd34..a9355f7a492 100644
--- a/parser/sql/dialect/oracle/src/main/java/org/apache/shardingsphere/sql/parser/oracle/visitor/statement/type/OracleDDLStatementVisitor.java
+++ b/parser/sql/dialect/oracle/src/main/java/org/apache/shardingsphere/sql/parser/oracle/visitor/statement/type/OracleDDLStatementVisitor.java
@@ -230,8 +230,6 @@ import org.apache.shardingsphere.sql.parser.statement.core.segment.procedure.Pro
import org.apache.shardingsphere.sql.parser.statement.core.segment.procedure.SQLStatementSegment;
import org.apache.shardingsphere.sql.parser.statement.core.statement.SQLStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterPackageStatement;
-import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterSessionStatement;
-import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterSystemStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AnalyzeStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AssociateStatisticsStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AuditStatement;
@@ -301,6 +299,8 @@ import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterOutl
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterPluggableDatabaseStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterProfileStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterRollbackSegmentStatement;
+import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterSessionStatement;
+import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterSystemStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleCreateClusterStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleCreateContextStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleCreateControlFileStatement;
@@ -880,7 +880,7 @@ public final class OracleDDLStatementVisitor extends OracleStatementVisitor impl
@Override
public ASTNode visitAlterSession(final AlterSessionContext ctx) {
- return new AlterSessionStatement();
+ return new OracleAlterSessionStatement();
}
@Override
@@ -890,7 +890,7 @@ public final class OracleDDLStatementVisitor extends OracleStatementVisitor impl
@Override
public ASTNode visitAlterSystem(final AlterSystemContext ctx) {
- return new AlterSystemStatement();
+ return new OracleAlterSystemStatement();
}
@Override
diff --git a/parser/sql/statement/core/src/main/java/org/apache/shardingsphere/sql/parser/statement/core/statement/type/ddl/AlterSystemStatement.java b/parser/sql/statement/type/oracle/src/main/java/org/apache/shardingsphere/sql/parser/statement/oracle/ddl/OracleAlterSessionStatement.java
similarity index 75%
rename from parser/sql/statement/core/src/main/java/org/apache/shardingsphere/sql/parser/statement/core/statement/type/ddl/AlterSystemStatement.java
rename to parser/sql/statement/type/oracle/src/main/java/org/apache/shardingsphere/sql/parser/statement/oracle/ddl/OracleAlterSessionStatement.java
index 851de86f154..595e11c2373 100644
--- a/parser/sql/statement/core/src/main/java/org/apache/shardingsphere/sql/parser/statement/core/statement/type/ddl/AlterSystemStatement.java
+++ b/parser/sql/statement/type/oracle/src/main/java/org/apache/shardingsphere/sql/parser/statement/oracle/ddl/OracleAlterSessionStatement.java
@@ -15,10 +15,12 @@
* limitations under the License.
*/
-package org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl;
+package org.apache.shardingsphere.sql.parser.statement.oracle.ddl;
+
+import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.DDLStatement;
/**
- * Alter system statement.
+ * Alter session statement for Oracle.
*/
-public final class AlterSystemStatement extends DDLStatement {
+public final class OracleAlterSessionStatement extends DDLStatement {
}
diff --git a/parser/sql/statement/core/src/main/java/org/apache/shardingsphere/sql/parser/statement/core/statement/type/ddl/AlterSessionStatement.java b/parser/sql/statement/type/oracle/src/main/java/org/apache/shardingsphere/sql/parser/statement/oracle/ddl/OracleAlterSystemStatement.java
similarity index 75%
rename from parser/sql/statement/core/src/main/java/org/apache/shardingsphere/sql/parser/statement/core/statement/type/ddl/AlterSessionStatement.java
rename to parser/sql/statement/type/oracle/src/main/java/org/apache/shardingsphere/sql/parser/statement/oracle/ddl/OracleAlterSystemStatement.java
index 6f2ba009c01..29be518125d 100644
--- a/parser/sql/statement/core/src/main/java/org/apache/shardingsphere/sql/parser/statement/core/statement/type/ddl/AlterSessionStatement.java
+++ b/parser/sql/statement/type/oracle/src/main/java/org/apache/shardingsphere/sql/parser/statement/oracle/ddl/OracleAlterSystemStatement.java
@@ -15,10 +15,12 @@
* limitations under the License.
*/
-package org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl;
+package org.apache.shardingsphere.sql.parser.statement.oracle.ddl;
+
+import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.DDLStatement;
/**
- * Alter session statement.
+ * Alter system statement for Oracle.
*/
-public final class AlterSessionStatement extends DDLStatement {
+public final class OracleAlterSystemStatement extends DDLStatement {
}
diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/DDLStatementAssert.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/DDLStatementAssert.java
index 750ff06fcbd..5e7b83aa62b 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/DDLStatementAssert.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/DDLStatementAssert.java
@@ -19,8 +19,6 @@ package org.apache.shardingsphere.test.it.sql.parser.internal.asserts.statement.
import lombok.AccessLevel;
import lombok.NoArgsConstructor;
-import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterSessionStatement;
-import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterSystemStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AnalyzeStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AssociateStatisticsStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AuditStatement;
@@ -54,6 +52,8 @@ import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.vi
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.view.CreateViewStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.view.DropViewStatement;
import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.view.RefreshMatViewStmtStatement;
+import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterSessionStatement;
+import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterSystemStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleFlashbackTableStatement;
import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleNoAuditStatement;
import org.apache.shardingsphere.sql.parser.statement.postgresql.ddl.PostgreSQLNotifyStmtStatement;
@@ -166,10 +166,10 @@ public final class DDLStatementAssert {
DropIndexStatementAssert.assertIs(assertContext, (DropIndexStatement) actual, (DropIndexStatementTestCase) expected);
} else if (actual instanceof AlterSynonymStatement) {
AlterSynonymStatementAssert.assertIs(assertContext, (AlterSynonymStatement) actual, (AlterSynonymStatementTestCase) expected);
- } else if (actual instanceof AlterSessionStatement) {
- AlterSessionStatementAssert.assertIs(assertContext, (AlterSessionStatement) actual, (AlterSessionStatementTestCase) expected);
- } else if (actual instanceof AlterSystemStatement) {
- AlterSystemStatementAssert.assertIs(assertContext, (AlterSystemStatement) actual, (AlterSystemStatementTestCase) expected);
+ } else if (actual instanceof OracleAlterSessionStatement) {
+ AlterSessionStatementAssert.assertIs(assertContext, (OracleAlterSessionStatement) actual, (AlterSessionStatementTestCase) expected);
+ } else if (actual instanceof OracleAlterSystemStatement) {
+ AlterSystemStatementAssert.assertIs(assertContext, (OracleAlterSystemStatement) actual, (AlterSystemStatementTestCase) expected);
} else if (actual instanceof AnalyzeStatement) {
AnalyzeStatementAssert.assertIs(assertContext, (AnalyzeStatement) actual, (AnalyzeStatementTestCase) expected);
} else if (actual instanceof AssociateStatisticsStatement) {
diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSessionStatementAssert.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSessionStatementAssert.java
index 9c41a06d847..d7000721cfa 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSessionStatementAssert.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSessionStatementAssert.java
@@ -19,7 +19,7 @@ package org.apache.shardingsphere.test.it.sql.parser.internal.asserts.statement.
import lombok.AccessLevel;
import lombok.NoArgsConstructor;
-import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterSessionStatement;
+import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterSessionStatement;
import org.apache.shardingsphere.test.it.sql.parser.internal.asserts.SQLCaseAssertContext;
import org.apache.shardingsphere.test.it.sql.parser.internal.cases.parser.jaxb.statement.ddl.AlterSessionStatementTestCase;
@@ -36,6 +36,6 @@ public final class AlterSessionStatementAssert {
* @param actual actual alter session statement
* @param expected expected alter session statement test case
*/
- public static void assertIs(final SQLCaseAssertContext assertContext, final AlterSessionStatement actual, final AlterSessionStatementTestCase expected) {
+ public static void assertIs(final SQLCaseAssertContext assertContext, final OracleAlterSessionStatement actual, final AlterSessionStatementTestCase expected) {
}
}
diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSystemStatementAssert.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSystemStatementAssert.java
index 3f97eadd0dc..ebb17da4e80 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSystemStatementAssert.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/internal/asserts/statement/ddl/impl/AlterSystemStatementAssert.java
@@ -19,7 +19,7 @@ package org.apache.shardingsphere.test.it.sql.parser.internal.asserts.statement.
import lombok.AccessLevel;
import lombok.NoArgsConstructor;
-import org.apache.shardingsphere.sql.parser.statement.core.statement.type.ddl.AlterSystemStatement;
+import org.apache.shardingsphere.sql.parser.statement.oracle.ddl.OracleAlterSystemStatement;
import org.apache.shardingsphere.test.it.sql.parser.internal.asserts.SQLCaseAssertContext;
import org.apache.shardingsphere.test.it.sql.parser.internal.cases.parser.jaxb.statement.ddl.AlterSystemStatementTestCase;
@@ -36,6 +36,6 @@ public final class AlterSystemStatementAssert {
* @param actual actual alter system statement
* @param expected expected alter system statement test case
*/
- public static void assertIs(final SQLCaseAssertContext assertContext, final AlterSystemStatement actual, final AlterSystemStatementTestCase expected) {
+ public static void assertIs(final SQLCaseAssertContext assertContext, final OracleAlterSystemStatement actual, final AlterSystemStatementTestCase expected) {
}
}