Copilot commented on code in PR #9267:
URL: https://github.com/apache/seatunnel/pull/9267#discussion_r2074804701
##########
seatunnel-connectors-v2/connector-jdbc/src/main/java/org/apache/seatunnel/connectors/seatunnel/jdbc/internal/dialect/oceanbase/OceanBaseMysqlDialect.java:
##########
@@ -137,6 +137,7 @@ public String extractTableName(TablePath tablePath) {
public Map<String, String> defaultParameter() {
HashMap<String, String> map = new HashMap<>();
map.put("rewriteBatchedStatements", "true");
Review Comment:
[nitpick] Consider adding a comment explaining why the "allowMultiQueries"
parameter is necessary for clarity and future maintainability.
```suggestion
map.put("rewriteBatchedStatements", "true");
// Enable multiple SQL queries in a single statement. This is
required for certain
// OceanBase MySQL operations that rely on executing multiple
queries together.
// Note: Use with caution as it may increase the risk of SQL
injection if inputs are not sanitized.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
