HyukjinKwon commented on code in PR #39214:
URL: https://github.com/apache/spark/pull/39214#discussion_r1057212456


##########
python/pyspark/sql/tests/test_catalog.py:
##########
@@ -16,20 +16,21 @@
 #

Review Comment:
   The changes made in reused PySpark tests are two:
   
   1. Do not check the specific type of exception. We should fix it to check 
properly (SPARK-41715)
   2. Explicit `collect` on `sql` because `sql` in Spark Connect is lazy (which 
actually I think is an issue).



-- 
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-unsubscr...@spark.apache.org

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