jorisvandenbossche commented on code in PR #49247:
URL: https://github.com/apache/arrow/pull/49247#discussion_r3020745827
##########
python/pyarrow/tests/test_pandas.py:
##########
@@ -2975,7 +2975,9 @@ def check_zero_copy_failure(self, arr):
arr.to_pandas(zero_copy_only=True)
def test_zero_copy_failure_on_object_types(self):
- self.check_zero_copy_failure(pa.array(['A', 'B', 'C']))
+ if Version(pd.__version__) < Version("3.0.0"):
+ # pandas 3.0 includes default string dtype support
Review Comment:
Yes, from what I can see this is an expected change, since string conversion
will now actually be zero copy
(although, strictly speaking, it is not actually zero-copy entirely, because
the test here is using string, and pandas will convert that to large_string.
But I suppose that happens outside the view of 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]
