itholic commented on code in PR #41660:
URL: https://github.com/apache/spark/pull/41660#discussion_r1233948214
##########
python/pyspark/pandas/tests/connect/test_parity_ewm.py:
##########
@@ -22,11 +22,15 @@
class EWMParityTests(EWMTestsMixin, PandasOnSparkTestUtils,
ReusedConnectTestCase, TestUtils):
- @unittest.skip("TODO(SPARK-43624): Enable ExponentialMovingLike.mean with
Spark Connect.")
+ @unittest.skip(
+ "TODO(SPARK-43611): Fix unexpected `AnalysisException` from Spark
Connect client."
+ )
Review Comment:
Before this fix, this test was JVM-dependent, but now it is generating the
same error as the unexpected `AnalysisException` posted in SPARK-43611. Will
address the related tickets collectively in SPARK-43611 at once.
##########
python/pyspark/pandas/tests/connect/test_parity_ewm.py:
##########
@@ -22,11 +22,15 @@
class EWMParityTests(EWMTestsMixin, PandasOnSparkTestUtils,
ReusedConnectTestCase, TestUtils):
- @unittest.skip("TODO(SPARK-43624): Enable ExponentialMovingLike.mean with
Spark Connect.")
+ @unittest.skip(
+ "TODO(SPARK-43611): Fix unexpected `AnalysisException` from Spark
Connect client."
+ )
Review Comment:
Before this fix, this test was JVM-dependent, but now it is generating the
same error as the unexpected `AnalysisException` posted in SPARK-43611. Will
address the related tickets collectively in SPARK-43611 at once.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
