Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20839#discussion_r176301672
  
    --- 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 to make the exception change look clean? I think it's fine to 
leave as is for now. I guess here we'll lose the information about exactly 
where the exception happened in Python 3 if I haven't missed something.


---

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

Reply via email to