AdamGS commented on code in PR #8822:
URL: https://github.com/apache/arrow-rs/pull/8822#discussion_r2515224471
##########
arrow-pyarrow-integration-testing/tests/test_sql.py:
##########
@@ -112,8 +114,16 @@ def assert_pyarrow_leak():
),
]
-_unsupported_pyarrow_types = [
-]
+if PYARROW_MAJOR_VER >= 16:
Review Comment:
Did this to get everything green, but I think that it might be a good
opportunity to only test with pyarrow 16 and above, which supports all relevant
features here - `stringview`, `bianryview` and `listview` and the `pycapsule`
API.
By the time this change (if accepted) will be released, pyarrow 15 will be
almost two years old, which seems like enough time to drop dedicated testing
for it. This will allow the CI to focus on more relevant releases that also
include all the APIs `arrow-rs` supports.
--
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]
