HeartSaVioR opened a new pull request #30167:
URL: https://github.com/apache/spark/pull/30167


   ### What changes were proposed in this pull request?
   
   This patch proposes to change the behavior on failing fast when Spark fails 
to instantiate configured v2 session catalog.
   
   ### Why are the changes needed?
   
   The Spark behavior is against the intention of the end users - if end users 
configure session catalog which Spark would fail to initialize, Spark would 
swallow the error with only logging the error message and silently use the 
default catalog implementation.
   
   This follows the voices on [discussion 
thread](https://lists.apache.org/thread.html/rdfa22a5ebdc4ac66e2c5c8ff0cd9d750e8a1690cd6fb456d119c2400%40%3Cdev.spark.apache.org%3E)
 in dev mailing list.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. After the PR Spark will fail immediately if Spark fails to instantiate 
configured session catalog.
   
   ### How was this patch tested?
   
   New UT added.


----------------------------------------------------------------
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:
us...@infra.apache.org



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

Reply via email to