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