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 2023/05/11 14:54:24 UTC

[shardingsphere] branch master updated: Fix sonar issue for ExternalMySQLTestParameterLoader (#25588)

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 f21de814d4e Fix sonar issue for ExternalMySQLTestParameterLoader (#25588)
f21de814d4e is described below

commit f21de814d4efe73e616ab6c551b001b7ca87811b
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Thu May 11 22:54:17 2023 +0800

    Fix sonar issue for ExternalMySQLTestParameterLoader (#25588)
---
 .../it/sql/parser/external/loader/ExternalMySQLTestParameterLoader.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/loader/ExternalMySQLTestParameterLoader.java b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/loader/ExternalMySQLTestParameterLoader.java
index 482b1596cbd..59dfab2548d 100644
--- a/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/loader/ExternalMySQLTestParameterLoader.java
+++ b/test/it/parser/src/main/java/org/apache/shardingsphere/test/it/sql/parser/external/loader/ExternalMySQLTestParameterLoader.java
@@ -50,7 +50,7 @@ public final class ExternalMySQLTestParameterLoader extends AbstractTestParamete
             if (line.isEmpty() || lines.isEmpty() && isComment(line)) {
                 continue;
             }
-            if (0 == lines.size() && line.toUpperCase().startsWith("DELIMITER")) {
+            if (lines.isEmpty() && line.toUpperCase().startsWith("DELIMITER")) {
                 delimiter = getNewDelimiter(line, delimiter);
                 continue;
             }