shardingsphere icon indicating copy to clipboard operation
shardingsphere copied to clipboard

Print error SQL when throwing exception.SQLParsingException

Open ninjaboy97 opened this issue 2 years ago • 4 comments

Fixes #18455 .

Changes proposed in this pull request:

  • Throw root cause of SQLParsingException

ninjaboy97 avatar Jun 25 '22 19:06 ninjaboy97

Codecov Report

Merging #18598 (590f1c9) into master (ed72b89) will decrease coverage by 0.00%. The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master   #18598      +/-   ##
============================================
- Coverage     59.32%   59.31%   -0.01%     
  Complexity     2255     2255              
============================================
  Files          3717     3717              
  Lines         54724    54726       +2     
  Branches       9326     9326              
============================================
  Hits          32463    32463              
- Misses        19510    19512       +2     
  Partials       2751     2751              
Impacted Files Coverage Δ
...advanced/AdvancedDistSQLStatementParserEngine.java 0.00% <0.00%> (ø)
...featured/FeaturedDistSQLStatementParserEngine.java 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ed72b89...590f1c9. Read the comment docs.

codecov-commenter avatar Jun 26 '22 04:06 codecov-commenter

Hey! I'll be able to update this by the coming weekend.

ninjaboy97 avatar Jul 05 '22 07:07 ninjaboy97

@ninjaboy97 Hi, any update?

terrymanu avatar Jul 20 '22 03:07 terrymanu

@ninjaboy97 The progress of this PR has not been upgraded for a long time. If there is no update, I will close it soon.

terrymanu avatar Aug 07 '22 16:08 terrymanu

I just close this PR and reopen issue #18455.

terrymanu avatar Aug 12 '22 04:08 terrymanu