-
Notifications
You must be signed in to change notification settings - Fork 6.9k
Add GraalVM Reachability Metadata and corresponding nativeTest for Presto Iceberg Connector #35504
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
Merged
Conversation
This file contains hidden or 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
d95f549 to
2a0d900
Compare
Closed
7 tasks
2a0d900 to
7709bfd
Compare
cf98540 to
8b107fe
Compare
…esto Iceberg Connector
8b107fe to
db6cbc7
Compare
strongduanmu
approved these changes
May 25, 2025
7 tasks
theNorthWindBlow
pushed a commit
to theNorthWindBlow/shardingsphere
that referenced
this pull request
May 26, 2025
…esto Iceberg Connector (apache#35504)
strongduanmu
pushed a commit
that referenced
this pull request
Aug 5, 2025
* Support parsing MySQL stored procedure syntax part3 * Add GraalVM Reachability Metadata and corresponding nativeTest for Presto Iceberg Connector (#35504) * Fix binding method in ProxyBackendHandlerFactory to include database type (#35506) * Add doc for Presto optional plugin (#35505) * Fix binding method in ProxyBackendHandlerFactory to include database type (#35507) * Refactor MySQLComStmtPrepareExecutor to use a class-level databaseType variable (#35508) * Optimize MySQLComFieldListPacketExecutor with DatabaseType (#35509) - Add a private final DatabaseType field initialized with "MySQL" - Replace inline DatabaseType retrieval with the new field for better readability and performance * Optimize MySQL multi-statements handling (#35510) - Extract DatabaseType instance as a private field to improve performance - Update SQLParserEngine initialization to use the pre-loaded DatabaseType - Modify SQLBindEngine invocation to include DatabaseType as a parameter * Add database type parameter to SQLBindEngine.bind method (#35511) * Optimize MySQL multi-statements handling - Extract DatabaseType instance as a private field to improve performance - Update SQLParserEngine initialization to use the pre-loaded DatabaseType - Modify SQLBindEngine invocation to include DatabaseType as a parameter * Add database type parameter to SQLBindEngine.bind method - Update the SQLBindEngine.bind method call to include the databaseType parameter - This change ensures that the database type is properly passed during SQL statement binding * Refactor PostgreSQLComParseExecutor to utilize DatabaseType in SQL parser engine creation (#35512) * Refactor PostgreSQLComDescribeExecutor to use ShardingSphereMetaData for SQL statement binding (#35513) * Support set datasource properties type with java.time.Duration (#34667) (#35241) * Fix PostgreSQLComDescribeExecutorTest (#35514) * Refactor PostgreSQLComDescribeExecutor to use ShardingSphereMetaData for SQL statement binding * Fix PostgreSQLComDescribeExecutorTest * Update release notes (#35515) * Refactor PostgreSQLBatchedStatementsExecutor to use database type consistently (#35516) * Refactor MySQLComFieldListPacketExecutor to use database type consistently (#35517) * Refactor PostgreSQLBatchedStatementsExecutor to use database type consistently * Refactor MySQLComFieldListPacketExecutor to use database type consistently * Refactor MySQLComFieldListPacketExecutor to use database type consistently * Refactor UnicastResourceShowExecutor to use database type consistently (#35518) * Refactor PreviewExecutor to use database type consistently (#35519) * Refactor SQL parsing to use DatabaseType consistently on ShardingSpherePreparedStatement and ShardingSphereStatement (#35520) * Add database type parameter to SQLBindEngine.bind method (#35521) - Update SQLBindEngine.bind method signature to include DatabaseType parameter - Modify test cases in various modules to pass DatabaseType to the bind method - This change enables more precise handling of database-specific SQL statements * Update RELEASE-NOTES.md This update is for Support parsing MySQL stored procedure syntax-part 3 #35503 --------- Co-authored-by: zhangyuanxing <[email protected]> Co-authored-by: Ling Hengqian <[email protected]> Co-authored-by: Liang Zhang <[email protected]> Co-authored-by: null <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
For #29052 .
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.