-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Closed
Closed
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
c5779ee
Support netty parameter ChannelOption.SO_BACKLOG configurable (#17812)
galaxylqx ef78ed3
Merge branch 'apache:master' into master
galaxylqx 3cc3842
Merge branch 'apache:master' into master
galaxylqx cb215e7
Merge branch 'apache:master' into master
galaxylqx 29dbd20
Merge branch 'apache:master' into master
galaxylqx 41da76a
Merge branch 'apache:master' into master
galaxylqx c5055ac
Merge branch 'apache:master' into master
galaxylqx 51629b7
Merge branch 'apache:master' into master
galaxylqx 2a99db7
Merge branch 'apache:master' into master
galaxylqx bf7a883
Merge branch 'apache:master' into master
galaxylqx ae27e33
Merge branch 'apache:master' into master
galaxylqx c56702b
Merge branch 'apache:master' into master
galaxylqx dee7a24
Merge branch 'apache:master' into master
galaxylqx 959a895
Merge branch 'apache:master' into master
galaxylqx d27cba6
Merge branch 'apache:master' into master
galaxylqx 4012601
Merge branch 'apache:master' into master
galaxylqx 4de4204
Merge branch 'apache:master' into master
galaxylqx 1683f33
Merge branch 'apache:master' into master
galaxylqx 81c3d13
Merge branch 'apache:master' into master
galaxylqx 3aee5c0
Merge branch 'apache:master' into master
galaxylqx e19e9c8
Merge branch 'apache:master' into master
galaxylqx f0c03b5
Merge branch 'apache:master' into master
galaxylqx 8f52964
Merge branch 'apache:master' into master
galaxylqx a9e0485
Merge branch 'apache:master' into master
galaxylqx c4abb9e
Merge branch 'apache:master' into master
galaxylqx 8034f27
Merge branch 'apache:master' into master
galaxylqx b6fbfbe
Merge branch 'apache:master' into master
galaxylqx d4ead85
Merge branch 'apache:master' into master
galaxylqx 48ed778
Merge branch 'apache:master' into master
galaxylqx 0a15200
Merge branch 'apache:master' into master
galaxylqx 06af4ae
Merge branch 'apache:master' into master
galaxylqx 6dce8d6
Merge branch 'apache:master' into master
galaxylqx a76ec92
Merge branch 'apache:master' into master
galaxylqx af16526
Merge branch 'apache:master' into master
galaxylqx 08f7e95
Merge branch 'apache:master' into master
galaxylqx 9a74856
Merge branch 'apache:master' into master
galaxylqx a70689b
Merge branch 'apache:master' into master
galaxylqx f214f5f
Merge branch 'apache:master' into master
galaxylqx 6ded5b1
Merge branch 'apache:master' into master
galaxylqx d34f8fb
Merge branch 'apache:master' into master
galaxylqx 1b79231
Merge branch 'apache:master' into master
galaxylqx 50bf89c
Merge branch 'apache:master' into master
galaxylqx 1b5f6b4
Merge branch 'apache:master' into master
galaxylqx 000f341
Merge branch 'apache:master' into master
galaxylqx d115cd8
Merge branch 'apache:master' into master
galaxylqx d1eb97d
Merge branch 'apache:master' into master
galaxylqx 69fbdb6
Merge branch 'apache:master' into master
galaxylqx b12afb3
Support SQL non-parsing, transparent transmission mode
galaxylqx File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -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 commentThe reason will be displayed to describe this comment to others. Learn more. Please add |
||
} | ||
|
||
private CacheOption createCacheOption(final CacheOption cacheOption, final CacheOptionSegment segment) { | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,10 +17,12 @@ | |
|
||
package org.apache.shardingsphere.sql.parser.sql.common.statement; | ||
|
||
import org.apache.shardingsphere.sql.parser.sql.dialect.statement.mysql.MySQLStatement; | ||
|
||
/** | ||
* 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 commentThe reason will be displayed to describe this comment to others. Learn more. How about rename it as |
||
|
||
@Override | ||
public int getParameterCount() { | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.