somiljain2006 opened a new pull request, #8124:
URL: https://github.com/apache/incubator-seata/pull/8124

   <!-- Please make sure you have read and understood the contributing 
guidelines -->
   
   - [x] I have read the 
[CONTRIBUTING.md](https://github.com/apache/incubator-seata/blob/2.x/CONTRIBUTING.md)
 guidelines.
   - [ ] I have registered the PR 
[changes](https://github.com/apache/incubator-seata/tree/2.x/changes).
   
   ### Ⅰ. Describe what this PR did
   
   Track placeholders inside SQL method expressions when resolving 
PreparedStatement parameter indexes for INSERT statements. This prevents 
incorrect primary key extraction when functions such as 
ST_GeomFromText(CONCAT(...)) contain embedded placeholders.
   
   ### Ⅱ. Does this pull request fix one issue?
   
   Fixes #6941 
   
   ### Ⅲ. Why don't you add test cases (unit test/integration test)? 
   
   Add regression tests covering SQL method expressions with nested 
placeholders.
   
   ### Ⅳ. Describe how to verify it
   
   mvn clean test
   
   ### Ⅴ. Special notes for reviews
   
   N/A


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to