Github user brennonyork commented on the pull request:
https://github.com/apache/spark/pull/5694#issuecomment-108956716
@pwendell thanks for the review! You're certainly correct in that I took a
"just get it into Python and working first" approach. Was unsure whether we
wanted more of what you had laid out above or something that got it into Python
and then incrementally built upon, but glad that clarification is there now.
Will move forward fixing the comments you and @JoshRosen supplied and get a new
commit back soon!
---
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]
