viirya commented on pull request #29497: URL: https://github.com/apache/spark/pull/29497#issuecomment-679549416
This idea sounds okay. I just have few questions. 1. Is it only for certain exceptions or all exceptions? For example, this PR targets only `AnalysisException`. 2. How to organize them? query compilation/query execution + package seems okay, but is there possibly unclear cases? For example, `AnalysisException` is also thrown in `RunnableCommand`, is it query compilation error and query execution error? 3. Is a top level object of error message too big to hold all exceptions in the module? For example `org.apache.spark.sql.QueryCompilationErrors` and `org.apache.spark.sql.QueryExecutionErrors` might contain many exceptions. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
