Skip to content

Commit d61d800

Browse files
authored
feat: update sql parse pattern (#558)
1 parent 2ddae4f commit d61d800

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

Diff for: arex-instrumentation-api/src/main/java/io/arex/inst/runtime/util/DatabaseUtils.java

+2-3
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,8 @@ public static String regenerateOperationName(String dbName, String operationName
3636
TableSchema tableSchema = JSqlParserUtil.parse(sql);
3737
tableSchema.setDbName(dbName);
3838
operationNames.add(regenerateOperationName(tableSchema, operationName));
39-
} catch (Throwable e) {
40-
// may be thrown error
41-
LogManager.warn("parse sql error", StringUtil.format("sql: %s", sql), e);
39+
} catch (Throwable ignore) {
40+
// ignore parse failure
4241
}
4342
}
4443
if (CollectionUtil.isEmpty(operationNames)) {

Diff for: arex-third-party/src/main/java/io/arex/agent/thirdparty/util/sqlparser/JSqlParserUtil.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
public class JSqlParserUtil {
1212

13-
private static final Pattern PATTERN = Pattern.compile("(\\s+|\"\\?\")");
13+
private static final Pattern PATTERN = Pattern.compile("(\\s+|\"\\?\"|\\[|\\])");
1414

1515
/**
1616
* parse table and action from sql

0 commit comments

Comments
 (0)