Github user marmbrus commented on the pull request:

    https://github.com/apache/spark/pull/11632#issuecomment-199421072
  
    I did a quick pass over this, and it seems reasonable.  However, this is a 
pretty big change so we should make sure we aren't missing anything.  It would 
be very helpful if you could augment the description with some more examples, 
focusing on the following:
     - code that used to run, but will now throw an exception
     - cases that didn't work before but do now
     - case where we still can't resolve as the user expects (but are hopefully 
giving a better error message)
    
    You might even look through JIRA/the user list to help enumerate cases.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to