itholic commented on code in PR #44778:
URL: https://github.com/apache/spark/pull/44778#discussion_r1470582255
##########
python/pyspark/sql/tests/test_utils.py:
##########
@@ -969,6 +984,7 @@ def test_assert_error_pandas_pyspark_df(self):
},
)
+ @unittest.skipIf(not have_pandas or not have_pyarrow, "no pandas or
pyarrow dependency")
def test_assert_error_non_pyspark_df(self):
Review Comment:
I believe we can remove `@unittest.skipIf(not have_pandas or not
have_pyarrow, "no pandas or pyarrow dependency")` as `assertDataFrameEqual` no
more requires `pandas` and `pyarrow`
--
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]
