You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by pa...@apache.org on 2023/05/17 08:06:10 UTC

[shardingsphere] branch master updated: Fix sonar issues on EspressoInlineExpressionParser (#25731)

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 c973093f468 Fix sonar issues on EspressoInlineExpressionParser (#25731)
c973093f468 is described below

commit c973093f468f3c48b4e3d6d457e70ea42643a1c4
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Wed May 17 16:06:02 2023 +0800

    Fix sonar issues on EspressoInlineExpressionParser (#25731)
---
 .../infra/expr/espresso/EspressoInlineExpressionParser.java        | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/infra/expr/espresso/src/main/java/org/apache/shardingsphere/infra/expr/espresso/EspressoInlineExpressionParser.java b/infra/expr/espresso/src/main/java/org/apache/shardingsphere/infra/expr/espresso/EspressoInlineExpressionParser.java
index 23b3f0b7168..811f79bf612 100644
--- a/infra/expr/espresso/src/main/java/org/apache/shardingsphere/infra/expr/espresso/EspressoInlineExpressionParser.java
+++ b/infra/expr/espresso/src/main/java/org/apache/shardingsphere/infra/expr/espresso/EspressoInlineExpressionParser.java
@@ -25,6 +25,7 @@ import org.graalvm.polyglot.Context;
 import org.graalvm.polyglot.TypeLiteral;
 import org.graalvm.polyglot.Value;
 
+import java.io.File;
 import java.net.URL;
 import java.util.ArrayList;
 import java.util.List;
@@ -44,10 +45,8 @@ public final class EspressoInlineExpressionParser implements InlineExpressionPar
         JAVA_HOME = System.getenv("JAVA_HOME");
         URL resource = EspressoInlineExpressionParser.class.getClassLoader().getResource("espresso-need-libs");
         String dir = null != resource ? resource.getPath() : null;
-        JAVA_CLASSPATH = Stream.of("groovy.jar", "guava.jar", "shardingsphere-infra-expr-hotsopt.jar",
-                "shardingsphere-infra-expr-spi.jar", "shardingsphere-infra-util.jar")
-                .map(s -> dir + "/" + s)
-                .collect(Collectors.joining(":"));
+        JAVA_CLASSPATH = Stream.of("groovy.jar", "guava.jar", "shardingsphere-infra-expr-hotsopt.jar", "shardingsphere-infra-expr-spi.jar", "shardingsphere-infra-util.jar")
+                .map(each -> dir + File.separator + each).collect(Collectors.joining(":"));
     }
     
     @Override