Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20839#discussion_r176302686
--- Diff: python/pyspark/sql/utils.py ---
@@ -121,7 +121,10 @@ def require_minimum_pandas_version():
from distutils.version import LooseVersion
try:
import pandas
+ have_pandas = True
except ImportError:
+ have_pandas = False
+ if not have_pandas:
--- End diff --
So, this is for making the error message clean when it's reraised in Python
3? I think it's fine to leave it as is. I believe the downside of it is to lose
the information where exactly the error was thrown in Python 3.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
