You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "HyukjinKwon (via GitHub)" <gi...@apache.org> on 2024/02/28 23:56:00 UTC

[PR] [SPARK-47202][PYTHON][TESTS][FOLLOW-UP] Run the test only with Python 3.9+ [spark]

HyukjinKwon opened a new pull request, #45324:
URL: https://github.com/apache/spark/pull/45324

   ### What changes were proposed in this pull request?
   
   This PR proposes to run the tzinfo test only with Python 3.9+. This is a followup of https://github.com/apache/spark/pull/45308.
   
   ### Why are the changes needed?
   
   To make the Python build passing with Python 3.8. It fails as below:
   
   ```python
   Starting test(pypy3): pyspark.sql.tests.test_arrow (temp output: /__w/spark/spark/python/target/605c2e61-b7c8-4898-ac7b-1d86f495bd4f/pypy3__pyspark.sql.tests.test_arrow__qrwyvw4l.log)
   Traceback (most recent call last):
     File "/usr/local/pypy/pypy3.8/lib/pypy3.8/runpy.py", line 197, in _run_module_as_main
       return _run_code(code, main_globals, None,
     File "/usr/local/pypy/pypy3.8/lib/pypy3.8/runpy.py", line 87, in _run_code
       exec(code, run_globals)
     File "/__w/spark/spark/python/pyspark/sql/tests/test_arrow.py", line 26, in <module>
       from zoneinfo import ZoneInfo
   ModuleNotFoundError: No module named 'zoneinfo'
   ```
   
   https://github.com/apache/spark/actions/runs/8082492167/job/22083534905
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, test-only.
   
   ### How was this patch tested?
   
   CI in this PR should test it out.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No,
   


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [SPARK-47202][PYTHON][TESTS][FOLLOW-UP] Run the test only with Python 3.9+ [spark]

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #45324:
URL: https://github.com/apache/spark/pull/45324#issuecomment-1970258661

   Merged to master.


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [SPARK-47202][PYTHON][TESTS][FOLLOW-UP] Run the test only with Python 3.9+ [spark]

Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon closed pull request #45324: [SPARK-47202][PYTHON][TESTS][FOLLOW-UP] Run the test only with Python 3.9+
URL: https://github.com/apache/spark/pull/45324


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org