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/06/20 09:01:10 UTC

[GitHub] [shardingsphere] yuzhiping95 opened a new issue, #18455: Print error SQL when throwing exception.SQLParsingException

yuzhiping95 opened a new issue, #18455:
URL: https://github.com/apache/shardingsphere/issues/18455

   Hi community:
        Now,the ShardingSphere can print exception log when we want to excute error SQL.
        For example Cause: com.google.common.util.concurrent.UncheckedExecutionException: org.apache.shardingsphere.sql.parser.exception.SQLParsingException: You have an error in your SQL syntax.
   
       However, when throwing error sql exception, the shardingSphere cant print error sql needing to parser   to help developer modify sql. I hope the shardingSphere can have the feature. Now,"sql-show:true "cant achieve the goal.  
      My shardingSphere version is 5.0.0
   
   
   


-- 
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.apache.org

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


[GitHub] [shardingsphere] xuup commented on issue #18455: Print error SQL when throwing exception.SQLParsingException

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

   Could you please assign this issue to me?


-- 
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] terrymanu commented on issue #18455: Print error SQL when throwing exception.SQLParsingException

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

   Hi @ninjaboy97 the screenshot from MySQL cli to report error is necessary.
   I don't make sure the result when exception thrown.


-- 
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] ninjaboy97 commented on issue #18455: Print error SQL when throwing exception.SQLParsingException

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

   Hey @terrymanu, raised a PR for this. I wasn't sure what needs to be done in case of FeaturedDistSQLStatementParserEngine throwing a SQLParseException since any of the instances from FeaturedDistSQLStatementParserFacadeFactory could be throwing the exception. Let me know what needs to be done here. Can you please also approve the CI workflows?
   
   Also @xuup apologies, I started working on this and missed your comment before raising the PR.


-- 
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] strongduanmu closed issue #18455: Print error SQL when throwing exception.SQLParsingException

Posted by GitBox <gi...@apache.org>.
strongduanmu closed issue #18455: Print error SQL when throwing exception.SQLParsingException
URL: https://github.com/apache/shardingsphere/issues/18455


-- 
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] tianhao960 commented on issue #18455: Print error SQL when throwing exception.SQLParsingException

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

   hi,I would like to try this.


-- 
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] terrymanu commented on issue #18455: Print error SQL when throwing exception.SQLParsingException

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

   Hi community, this issue still open for pick up.


-- 
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] terrymanu commented on issue #18455: Print error SQL when throwing exception.SQLParsingException

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

   It is better to throw the root cause of SQL parser Exception. 
   @yuzhiping95 Do you interest to do the enhancement? 


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