Skip to content

Conversation

@linghengqian
Copy link
Member

For #29052.

Changes proposed in this pull request:


Before committing this PR, I'm sure that I have checked the following options:

  • My code follows the code of conduct of this project.
  • I have self-reviewed the commit code.
  • I have (or in comment I request) added corresponding labels for the pull request.
  • I have passed maven check locally : ./mvnw clean install -B -T1C -Dmaven.javadoc.skip -Dmaven.jacoco.skip -e.
  • I have made corresponding changes to the documentation.
  • I have added corresponding unit tests for my changes.
  • I have updated the Release Notes of the current development version. For more details, see Update Release Note

@linghengqian linghengqian marked this pull request as ready for review May 25, 2025 08:56
@linghengqian linghengqian added this to the 5.5.3 milestone May 25, 2025
@terrymanu terrymanu merged commit 7321d0d into apache:master May 25, 2025
148 checks passed
@linghengqian linghengqian deleted the presto-doc branch May 25, 2025 13:02
theNorthWindBlow pushed a commit to theNorthWindBlow/shardingsphere that referenced this pull request May 26, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants