Github user BryanCutler commented on the issue:

    https://github.com/apache/spark/pull/20839
  
    @HyukjinKwon and @ueshin I came close to a solution that would wrap the 
exception message like before but still keep the exception type and traceback.  
The problem was there are too many variations in python exceptions and it was 
getting too complicated to be foolproof.  For instance, `ImportError` does not 
use `args` for the message, but has another attribute `message` in Python 2 and 
`msg` in Python 3.
    
    To keep things simple for this, I thought it would be fine when fallback is 
disabled, to still print a warning message to indicate the related conf, and 
then use `raise` to re-raise the exception which retains the original exception 
with traceback.  I manually ran tests of toPandas and createDataFrame with 
Python 2 & 3 and it looks cleaner imo.  What do you guys think?


---

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

Reply via email to