From 575afffe2fe448c366874a067aafcab28d283058 Mon Sep 17 00:00:00 2001 From: etherge Date: Sun, 11 Feb 2018 12:00:07 +0800 Subject: [PATCH] KYLIN-3135 fix regEx issue --- query/src/main/java/org/apache/kylin/query/util/QueryUtil.java | 2 +- .../src/test/java/org/apache/kylin/query/util/QueryUtilTest.java | 9 ++++++++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java b/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java index da18bd7..7b6e49f 100644 --- a/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java +++ b/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java @@ -142,7 +142,7 @@ public class QueryUtil { public static String removeCommentInSql(String sql1) { // match two patterns, one is "-- comment", the other is "/* comment */" - final String[] commentPatterns = new String[] { "--.*?[\r\n]", "/\\*.*?\\*/" }; + final String[] commentPatterns = new String[] { "--.*?[\r\n]", "/\\*[\\s\\S]*?\\*/" }; for (int i = 0; i < commentPatterns.length; i++) { sql1 = sql1.replaceAll(commentPatterns[i], ""); diff --git a/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java b/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java index e3cd819..af0501a 100644 --- a/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java +++ b/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java @@ -164,7 +164,14 @@ public class QueryUtilTest extends LocalFileMetadataTestCase { } { - String sqlWithComment = "/* comment1 * comment2 */ -- comment 5\n" + originSql + "/* comment3 / comment4 */"; + String sqlWithComment = "/* comment1 * comment2 */ -- comment 5\n" + originSql + + "/* comment3 / comment4 */"; + Assert.assertEquals(originSql, QueryUtil.removeCommentInSql(sqlWithComment)); + } + + { + String sqlWithComment = "/* comment1 \r\n comment2 * \n comment1 */ -- comment 5\n" + originSql + + "/* comment3 / comment4 */"; Assert.assertEquals(originSql, QueryUtil.removeCommentInSql(sqlWithComment)); } } -- 2.9.3 (Apple Git-75)