Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/10787#issuecomment-174335545
Suggestion: we can merge the documentation fixes separately. I'm not
especially motivated to review these code changes if they're just internal
cleanup / refactoring, since this isn't a high-activity component and I'd
prefer to just leave known working code in place.
Therefore, here's my proposal:
- Close this PR and open a new one which contains only documentation fixes.
This can be merged quickly.
- If there are bugfixes / behavior changes, submit _only_ those changes as
separate small and easy to review PRs.
- Don't do the giant refactoring for refactoring's sake; it's not worth the
risk + review time.
---
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]
