terencehonles commented on a change in pull request #8386:
URL: https://github.com/apache/arrow/pull/8386#discussion_r501463131



##########
File path: dev/tasks/python-wheels/travis.osx.yml
##########
@@ -31,7 +31,7 @@ addons:
       - git
       - openssl@1.1
       - protobuf
-      - python@3.8
+      - python@3.9
       - thrift
       - wget
     update: false

Review comment:
       I think it's possible Travis doesn't have Python 3.9 in it's homebrew 
cache, but I would expect a different error than it succeeds and then 
`/usr/local/opt/python@3.8/libexec/bin/pip` is just not found...
   
   Nevermind! Apparently it's just a "warning" :unamused: :
   > Warning: 'python@3.9' formula is unreadable: No available formula with the 
name "python@3.9" 
   
   https://travis-ci.org/github/ursa-labs/crossbow/builds/733873746#L145

##########
File path: python/pyproject.toml
##########
@@ -19,7 +19,9 @@
 requires = [
     "cython >= 0.29",
     "numpy==1.14.5; python_version<'3.7'",
-    "numpy==1.16.0; python_version>='3.7'",

Review comment:
       Just to follow up, 1.19.2 did work for me, I'm just saying there might 
be a earlier release that works. It's probably simplest/best to use 1.19.2 
since it's mentioned in their changelog:
   
   > This release supports Python 3.6-3.8. Cython >= 0.29.21 needs to be used 
when building with Python 3.9 for testing purposes.
   https://numpy.org/devdocs/release/1.19.2-notes.html
   
   ...looking at 1.19.1 it says the same thing so it might be safe to go 
backwards




----------------------------------------------------------------
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.

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


Reply via email to