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