You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/07/28 10:29:58 UTC

[GitHub] [spark] beliefer commented on pull request #33535: [SPARK-36108][SQL] Refactor first set of 20 query parsing errors to use error classes

beliefer commented on pull request #33535:
URL: https://github.com/apache/spark/pull/33535#issuecomment-888200914


   > Thanks for working on this! I left some suggestions to simplify the error classes. We should also make sure the SQLSTATEs match the ISO standard; if you're not sure what SQLSTATE fits, you can also leave it empty for now.
   
   Can you tell me how to know the SQLSTATEs?


-- 
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: reviews-unsubscribe@spark.apache.org

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