New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support SQL non-parsing, transparent transmission mode #25180
Conversation
@@ -34,4 +34,6 @@ public final class SQLParserRuleConfiguration implements GlobalRuleConfiguration | |||
private final CacheOption parseTreeCache; | |||
|
|||
private final CacheOption sqlStatementCache; | |||
|
|||
private final String engineType; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add default value for engineType. It is better to add a compatible constructor.
@@ -40,7 +40,7 @@ public SQLParserRuleConfiguration buildAlteredRuleConfiguration(final SQLParserR | |||
CacheOption sqlStatementCache = | |||
null == sqlStatement.getSqlStatementCache() ? currentRuleConfig.getSqlStatementCache() | |||
: createCacheOption(currentRuleConfig.getSqlStatementCache(), sqlStatement.getSqlStatementCache()); | |||
return new SQLParserRuleConfiguration(sqlCommentParseEnabled, parseTreeCache, sqlStatementCache); | |||
return new SQLParserRuleConfiguration(sqlCommentParseEnabled, parseTreeCache, sqlStatementCache, "Standard"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add // TODO with reason
to reminder it fix in future
/** | ||
* Simple SQL statement. | ||
*/ | ||
public final class SimpleSQLStatement implements SQLStatement { | ||
public final class SimpleSQLStatement implements MySQLStatement { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about rename it as MySQLSimpleSQLStatement
and create real SimpleSQLStatement
?
Since this pr has long time no response, I will close it now. You are welcome to submit a new PR and modify the comment issue. |
Fixes #ISSUSE_ID.
Changes proposed in this pull request:
Before committing this PR, I'm sure that I have checked the following options:
./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e
.