itholic commented on code in PR #44881:
URL: https://github.com/apache/spark/pull/44881#discussion_r1466155995


##########
dev/infra/Dockerfile:
##########
@@ -91,10 +91,10 @@ RUN mkdir -p /usr/local/pypy/pypy3.8 && \
     ln -sf /usr/local/pypy/pypy3.8/bin/pypy /usr/local/bin/pypy3.8 && \
     ln -sf /usr/local/pypy/pypy3.8/bin/pypy /usr/local/bin/pypy3
 RUN curl -sS https://bootstrap.pypa.io/get-pip.py | pypy3
-RUN pypy3 -m pip install numpy 'six==1.16.0' 'pandas<=2.1.4' scipy coverage 
matplotlib lxml
+RUN pypy3 -m pip install numpy 'six==1.16.0' 'pandas<=2.2.0' scipy coverage 
matplotlib lxml

Review Comment:
   AFAIK, pip automatically finds the most recent version that meets the 
conditions as below (Has this not worked well so far btw??):
   
   ```
   (pyspark-dev-env) spark % pip install "pandas<=2.2.0"
   Collecting pandas<=2.2.0
   ...
   Installing collected packages: pandas
   Successfully installed pandas-2.2.0
   ```
   
   But I'm okay with the way `==`. WDYT, @HyukjinKwon ?
   



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

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


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

Reply via email to