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

chengzhang 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 9e329a824f7 Fix sonar issue in EnumerableScan (#26772)
9e329a824f7 is described below

commit 9e329a824f733952787bb1e6a74fea2447cf8d4f
Author: Zhengqiang Duan <[email protected]>
AuthorDate: Wed Jul 5 15:48:46 2023 +0800

    Fix sonar issue in EnumerableScan (#26772)
---
 .../sqlfederation/compiler/operator/physical/EnumerableScan.java     | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/compiler/operator/physical/EnumerableScan.java
 
b/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/compiler/operator/physical/EnumerableScan.java
index 38584be7cc7..b859a9cc383 100644
--- 
a/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/compiler/operator/physical/EnumerableScan.java
+++ 
b/kernel/sql-federation/core/src/main/java/org/apache/shardingsphere/sqlfederation/compiler/operator/physical/EnumerableScan.java
@@ -42,6 +42,7 @@ import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
+import java.util.regex.Pattern;
 
 /**
  * Enumerable scan.
@@ -49,6 +50,8 @@ import java.util.Objects;
 @Getter
 public final class EnumerableScan extends TableScan implements EnumerableRel {
     
+    private static final Pattern LINE_SEPARATOR_PATTERN = 
Pattern.compile("[\r\n]");
+    
     private final SqlString sqlString;
     
     private final RelDataType pushDownRowType;
@@ -74,7 +77,7 @@ public final class EnumerableScan extends TableScan 
implements EnumerableRel {
     
     @Override
     public RelWriter explainTerms(final RelWriter relWriter) {
-        return super.explainTerms(relWriter).item("sql", 
sqlString.getSql().replaceAll("[\r\n]", " "));
+        return super.explainTerms(relWriter).item("sql", 
LINE_SEPARATOR_PATTERN.matcher(sqlString.getSql()).replaceAll(" "));
     }
     
     @Override

Reply via email to