AlenkaF commented on code in PR #35546: URL: https://github.com/apache/arrow/pull/35546#discussion_r1191180662
########## python/pyarrow/tests/test_pandas.py: ########## @@ -4816,3 +4816,23 @@ def test_unhashable_map_keys_with_pydicts(): for tup1, tup2 in zip(row1, row2): assert np.array_equal(tup1[0], tup2[0]) assert tup1[1] == tup2[1] + + +def test_column_conversion_for_datetime(): + # GH-35235 + # pandas implemented __from_arrow__ for DatetimeTZDtype + # https://github.com/pandas-dev/pandas/pull/52201 + if Version(pd.__version__) <= Version("2.0.1"): + pytest.skip("__from_arrow__ added to DatetimeTZDtype in pandas version 2.1.0") Review Comment: Yes, you are correct! I will put the main part of the test outside this skip and leave `assert hasattr(pandas_dtype, '__from_arrow__')` in it. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org