damccorm commented on code in PR #36845: URL: https://github.com/apache/beam/pull/36845#discussion_r2592835606
########## sdks/python/tox.ini: ########## @@ -362,6 +362,10 @@ deps = 16: pyarrow>=16,<17 17: pyarrow>=17,<18 18: pyarrow>=18,<19 + 19: pyarrow>=19,<20 + 20: pyarrow>=20,<21 + 21: pyarrow>=21,<22 + 22: pyarrow>=22,<23 Review Comment: Could you please update https://github.com/apache/beam/blob/84684aef490e03163295050e3b6e51aa6c6a2ee6/sdks/python/test-suites/tox/py310/build.gradle#L88 to exercise these versions? That will ensure they get picked up by continuous integration ########## sdks/python/tox.ini: ########## Review Comment: As part of this, would you mind removing versions 10-14? This is consistent with the comment: `Test against versions of pyarrow released in last ~2 years.` -- 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]
