HyukjinKwon commented on code in PR #43184: URL: https://github.com/apache/spark/pull/43184#discussion_r1341950255
########## python/pyspark/pandas/plot/matplotlib.py: ########## @@ -15,7 +15,7 @@ # limitations under the License. # -from distutils.version import LooseVersion +from packaging.version import Version Review Comment: Oh actually we can't just add a required dependency easily. It works with `pip install` but it wouldn't work when users download Spark from the official website (as they have to manually install `packaging` dependency in their nodes) - in case of Py4J, we already contain that in our release so it works. We should either port `packaging` into our release, or find a workaround to avoid adding the required dependency. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
