spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [spark] beliefer opened a new pull request #30564: [SPARK-32670][SQL][FOLLOWUP] Group exception messages in Catalyst Analyzer in one file
Date Wed, 02 Dec 2020 06:18:16 GMT

beliefer opened a new pull request #30564:
URL: https://github.com/apache/spark/pull/30564


   ### What changes were proposed in this pull request?
   This PR follows up https://github.com/apache/spark/pull/29497.
   Because https://github.com/apache/spark/pull/29497 just give us an example to group all
`AnalysisExcpetion` in Analyzer into QueryCompilationErrors.
   This PR group other `AnalysisExcpetion` into QueryCompilationErrors.
   
   
   ### Why are the changes needed?
   It will largely help with standardization of error messages and its maintenance.
   
   
   ### Does this PR introduce _any_ user-facing change?
   No. Error messages remain unchanged.
   
   
   ### How was this patch tested?
   No new tests - pass all original tests to make sure it doesn't break any existing behavior.
   


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message