Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/20839#discussion_r176835709
--- 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 --
I think having the traceback to the `raise ImportError` below is all the
information needed. If that happens, then the only possible cause is that the
import failed from here. The problem with how it was before is that for Python
3, it will print out `During handling of the above exception, another exception
occurred:` which makes it seem like it is not being handled correctly, since
it's really just a failed import.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
