Skip to content
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

Fixed issues in the MySQL component where the executeBatch method could result in empty SQL statements . #702

Merged
merged 3 commits into from
Jul 1, 2024

Conversation

w2dp
Copy link
Contributor

@w2dp w2dp commented Jul 1, 2024

Fixed an issue in the MySQL component where the executeBatch method could obtain SQL in certain scenarios, but the final SQL statement was empty.
The issue arises because the existing code defaults to using the first parameter of the enhanced method as the source of the SQL statement. However, the executeBatch method does not have any parameters. Given the previous problem where SQL could not be obtained from the cacheObject, this approach merely attempts to retrieve the SQL to the best extent possible, though perfect collection cannot be guaranteed.

刘颖 added 3 commits July 1, 2024 14:30
Fixed an issue in the MySQL component where the executeBatch method
could obtain SQL in certain scenarios, but the final SQL statement was
empty.
Fixed an issue in the MySQL component where SQL parameters were
not collected in certain scenarios (e.g., executeBatch).
@w2dp w2dp changed the title Fixed issues in the MySQL component where the executeBatch method could result in empty SQL statements and missing SQL parameters in certain scenarios. Fixed issues in the MySQL component where the executeBatch method could result in empty SQL statements . Jul 1, 2024
@wu-sheng wu-sheng added enhancement New feature or request plugin labels Jul 1, 2024
@wu-sheng wu-sheng added this to the 9.3.0 milestone Jul 1, 2024
@wu-sheng wu-sheng merged commit 2e08217 into apache:main Jul 1, 2024
191 checks passed
@w2dp w2dp deleted the fix_mysql_executeBatch branch September 18, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants