HyukjinKwon commented on a change in pull request #25684: 
[SPARK-28980][CORE][SQL][STREAMING][MLLIB] Remove most items deprecated in 
Spark 2.2.0 or earlier, for Spark 3
URL: https://github.com/apache/spark/pull/25684#discussion_r321573454
 
 

 ##########
 File path: python/pyspark/sql/tests/test_context.py
 ##########
 @@ -40,15 +40,20 @@ def setUpClass(cls):
         ReusedPySparkTestCase.setUpClass()
         cls.tempdir = tempfile.NamedTemporaryFile(delete=False)
         cls.hive_available = True
+        cls.spark = None
         try:
             cls.sc._jvm.org.apache.hadoop.hive.conf.HiveConf()
 
 Review comment:
   @srowen, seems this code is still needed. 
`SparkSession.builder.enableHiveSupport().getOrCreate()` seems not able to 
directly check if Hive is available or not. I manually tested and just pushed 
the changes directly into your branch.

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


With regards,
Apache Git Services

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

Reply via email to