[GitHub] [shardingsphere] taojintianxia commented on pull request #24227: Remove hint dist sql

["taojintianxia (via GitHub)" <gi...@apache.org>]

taojintianxia commented on PR #24227:
URL: https://github.com/apache/shardingsphere/pull/24227#issuecomment-1439356651

   > > @anmui Yes, `proxy-hint-enabled` should be removed, it's just for Hint DistSQL. But after `proxy-hint-enabled` is deleted, the test transaction type is local and the connection will be reused. The E2E test will fails. 😭 I solve this problem by establishing a connection for each request even if the transaction type is local in the proxy. Can we solve this problem in this way? 😂 I guess it can be solved by changing the configuration of the test environment..🤔
   > > ```
   > >  transaction:
   > >     defaultType: XA
   > >     providerType: Atomikos
   > > ```
   > 
   > Hi @taojintianxia , can you take a look? Thank you!
   
   this seems not a environment issue, I suggest to take a look on the logic of the corresponding code.


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org