ueshin commented on a change in pull request #32386:
URL: https://github.com/apache/spark/pull/32386#discussion_r623531116



##########
File path: python/docs/source/getting_started/install.rst
##########
@@ -159,6 +159,9 @@ Package       Minimum supported version Note
 `NumPy`       1.7                       Required for ML 
 `pyarrow`     1.0.0                     Optional for SQL
 `Py4J`        0.10.9.2                  Required
+`pandas`      0.23.2                    Required for Pandas-on-spark
+`pyarrow`     0.10                      Required for Pandas-on-spark
+`Numpy`       1.14(<1.20.0)             Required for Pandas-on-spark

Review comment:
       `pandas-on-Spark`?

##########
File path: dev/requirements.txt
##########
@@ -6,3 +6,13 @@ pydata_sphinx_theme
 ipython
 nbsphinx
 numpydoc
+
+# dependencies in pandas-on-spark.
+pandas>=0.23.2
+pyarrow>=0.10
+numpy>=1.14,<1.20.0
+
+# Optional dependencies in pandas-on-spark.
+mlflow>=1.0
+plotly>=4.8
+matplotlib>=3.0.0,<3.3.0

Review comment:
       I'm not sure we should add them here because even dependencies for SQL 
or ML modules are not here.




-- 
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to