Github user JoshRosen commented on the pull request:

    https://github.com/apache/spark/pull/2659#issuecomment-62780140
  
    I guess this seems fine to me, since I think I reviewed it previously and 
it doesn't look like much has changed.  It would be nice if there were more 
explanatory comments so that someone who knows nothing about this JIRA / code 
can figure out what's going on.  On the other hand, PythonRDD already suffers 
from poor documentation, so I don't mind fixing this up myself in a subsequent 
pull request to add lots of comments / developer docs.
    
    Therefore, I'll probably merge this after Jenkins passes.


---
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