This is an automated email from the ASF dual-hosted git repository.

duanzhengqiang 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 dbd77a200e9 Rename joinTableSegments to joinTables. (#25462)
dbd77a200e9 is described below

commit dbd77a200e97594fe5693717a11df3e8d42b4184
Author: Chuxin Chen <[email protected]>
AuthorDate: Fri May 5 13:41:04 2023 +0800

    Rename joinTableSegments to joinTables. (#25462)
---
 .../sql/parser/sql/common/extractor/TableExtractor.java             | 6 +++---
 .../sql/parser/sql/common/extractor/TableExtractorTest.java         | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/sql-parser/statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractor.java
 
b/sql-parser/statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractor.java
index bf9d68ee10f..7571cb4747d 100644
--- 
a/sql-parser/statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractor.java
+++ 
b/sql-parser/statement/src/main/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractor.java
@@ -68,7 +68,7 @@ public final class TableExtractor {
     
     private final Collection<TableSegment> tableContext = new LinkedList<>();
     
-    private final Collection<JoinTableSegment> joinTableSegments = new 
LinkedList<>();
+    private final Collection<JoinTableSegment> joinTables = new LinkedList<>();
     
     /**
      * Extract table that should be rewritten from select statement.
@@ -110,10 +110,10 @@ public final class TableExtractor {
             TableExtractor tableExtractor = new TableExtractor();
             tableExtractor.extractTablesFromSelect(((SubqueryTableSegment) 
tableSegment).getSubquery().getSelect());
             rewriteTables.addAll(tableExtractor.rewriteTables);
-            joinTableSegments.addAll(tableExtractor.joinTableSegments);
+            joinTables.addAll(tableExtractor.joinTables);
         }
         if (tableSegment instanceof JoinTableSegment) {
-            joinTableSegments.add((JoinTableSegment) tableSegment);
+            joinTables.add((JoinTableSegment) tableSegment);
             extractTablesFromJoinTableSegment((JoinTableSegment) tableSegment);
         }
         if (tableSegment instanceof DeleteMultiTableSegment) {
diff --git 
a/sql-parser/statement/src/test/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractorTest.java
 
b/sql-parser/statement/src/test/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractorTest.java
index 9b69c0d8223..f05c6780d9e 100644
--- 
a/sql-parser/statement/src/test/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractorTest.java
+++ 
b/sql-parser/statement/src/test/java/org/apache/shardingsphere/sql/parser/sql/common/extractor/TableExtractorTest.java
@@ -184,7 +184,7 @@ class TableExtractorTest {
         MySQLSelectStatement selectStatement = new MySQLSelectStatement();
         selectStatement.setFrom(joinTableSegment);
         tableExtractor.extractTablesFromSelect(selectStatement);
-        assertThat(tableExtractor.getJoinTableSegments().size(), is(1));
-        assertThat(tableExtractor.getJoinTableSegments().iterator().next(), 
is(joinTableSegment));
+        assertThat(tableExtractor.getJoinTables().size(), is(1));
+        assertThat(tableExtractor.getJoinTables().iterator().next(), 
is(joinTableSegment));
     }
 }

Reply via email to