tvalentyn commented on code in PR #32429:
URL: https://github.com/apache/beam/pull/32429#discussion_r1755142346


##########
.test-infra/jenkins/PythonTestProperties.groovy:
##########
@@ -20,10 +20,10 @@ class PythonTestProperties {
   // Indicates all supported Python versions.
   // This must be sorted in ascending order.
   final static List<String> ALL_SUPPORTED_VERSIONS = [
-    '3.8',
     '3.9',

Review Comment:
   is this file still being used given that we don't use Jenkins? 



##########
sdks/python/tox.ini:
##########
@@ -271,7 +271,7 @@ commands =
   bash {toxinidir}/scripts/pytest_validates_runner.sh {envname} 
{toxinidir}/apache_beam/runners/portability/spark_runner_test.py {posargs}
 
 
-[testenv:py{38,39}-pyarrow-{3,9,10,11,12,13,14,15,16}]
+[testenv:py{39,310}-pyarrow-{3,9,10,11,12,13,14,15,16}]

Review Comment:
   keeping just 39 is also fine here, if 310 is not used anywhere 



-- 
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...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to