You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2022/01/09 10:14:21 UTC

[GitHub] [shardingsphere] userkdg opened a new issue #14636: shardingsphere sql parser support multi statements

userkdg opened a new issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636


   ## Feature Request
   
   ### Is your feature request related to a problem?
   app/client had some one request with mult sql , such as ,the mybatis xml 
   ``` xml 
   
   <update>
       <foreach collection="bos" item="item" split=";">
           update xx set name= #{item.name} where  id = #{item.id}
       </foreach>
   </update>
   ```
   
   ### Describe the feature you would like.
   on sharding-proxy,  accept one packet with multi sql, can foreach every sql packet to submitted.
   


-- 
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



[GitHub] [shardingsphere] userkdg commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
userkdg commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081293939


   need support :
   ![Uploading image.png…]()
   


-- 
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



[GitHub] [shardingsphere] TeslaCN edited a comment on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN edited a comment on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081417043


   The features may be implemented in future release. But this is not because of unsupported feature, insert with values may faster than multi statements. 


-- 
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



[GitHub] [shardingsphere] TeslaCN commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1013036533


   Hi @userkdg 
   Any update here?


-- 
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



[GitHub] [shardingsphere] TeslaCN commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1008616130


   Could you provide more information about this issue?


-- 
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



[GitHub] [shardingsphere] userkdg edited a comment on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
userkdg edited a comment on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081293939


   need support :
   <img width="694" alt="企业微信截图_16485156797884" src="https://user-images.githubusercontent.com/21057449/160511775-3a021b16-a6a7-4d8b-a10c-6a81f70e06da.png">
   


-- 
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



[GitHub] [shardingsphere] userkdg commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
userkdg commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081338819


   > > @TeslaCN one packet had mult sql from app, then proxy will throw exception ,such as :
   > > ```java
   > > Cause: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT INTO `ec_oms_config_order_hang_sub` (`hand_code`, `order_type`, `province' at line 1
   > > ```
   > > 
   > > 
   > >     
   > >       
   > >     
   > > 
   > >       
   > >     
   > > 
   > >     
   > >   
   > > ![image](https://user-images.githubusercontent.com/21057449/149514220-0c39677d-1a1a-4af6-b1fe-a7374167815e.png)
   > 
   > @userkdg It seems this exception was thrown by JDBC Driver instead of Proxy.
   
   This issue has been fixed locally, reason is :
   mybatis foreach insert/update... sql parser error and sharding-encrypt sql rewrite unsupport
   


-- 
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



[GitHub] [shardingsphere] TeslaCN commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081342180


   @userkdg You may consider doing batched inserts by multi values instead of multi statements.


-- 
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



[GitHub] [shardingsphere] TeslaCN commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081333422


   > @TeslaCN one packet had mult sql from app, then proxy will throw exception ,such as :
   > 
   > ```java
   > Cause: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT INTO `ec_oms_config_order_hang_sub` (`hand_code`, `order_type`, `province' at line 1
   > ```
   > 
   > ![image](https://user-images.githubusercontent.com/21057449/149514220-0c39677d-1a1a-4af6-b1fe-a7374167815e.png)
   
   @userkdg I seems this exception was thrown by JDBC Driver instead of Proxy.


-- 
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



[GitHub] [shardingsphere] userkdg commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
userkdg commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1013069937


   @TeslaCN one packet had mult sql from app, then proxy will throw exception ,such as :
   ``` java
   
   Cause: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT INTO `ec_oms_config_order_hang_sub` (`hand_code`, `order_type`, `province' at line 1
   
   ``` 
   ![image](https://user-images.githubusercontent.com/21057449/149514220-0c39677d-1a1a-4af6-b1fe-a7374167815e.png)
   
   
   
   


-- 
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



[GitHub] [shardingsphere] userkdg commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
userkdg commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081345387


   @TeslaCN It is not friendly to mature business systems. In addition, shouldn't we solve this problem from the framework?


-- 
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



[GitHub] [shardingsphere] TeslaCN edited a comment on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN edited a comment on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081333422


   > @TeslaCN one packet had mult sql from app, then proxy will throw exception ,such as :
   > 
   > ```java
   > Cause: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT INTO `ec_oms_config_order_hang_sub` (`hand_code`, `order_type`, `province' at line 1
   > ```
   > 
   > ![image](https://user-images.githubusercontent.com/21057449/149514220-0c39677d-1a1a-4af6-b1fe-a7374167815e.png)
   
   @userkdg It seems this exception was thrown by JDBC Driver instead of Proxy.


-- 
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



[GitHub] [shardingsphere] TeslaCN commented on issue #14636: shardingsphere sql parser support multi statements

Posted by GitBox <gi...@apache.org>.
TeslaCN commented on issue #14636:
URL: https://github.com/apache/shardingsphere/issues/14636#issuecomment-1081417043


   It is not because of unsupported feature, insert with values may faster than multi statements.


-- 
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