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