[GitHub] [spark] gengliangwang commented on pull request #35936: [SPARK-38616][SQL] Keep track of SQL query text in Catalyst TreeNode

[GitBox <gi...@apache.org>]

gengliangwang commented on pull request #35936:
URL: https://github.com/apache/spark/pull/35936#issuecomment-1075141041


   I am still deciding whether to show the whole original query text or just the problematic part in the runtime exception message. Thus I didn't change the error message of an `Expression` as an example in this PR. 
   
   cc @entong @cloud-fan let me know if you think we need an example and end-to-end test.


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