Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/2303#issuecomment-54718542
I'd still like for these to be combined into a single PR. Each additional
PR means another run of Jenkins tests, another commit message, another round of
emails / entry in our [PR tracker](http://spark-prs.appspot.com/); this adds up
to be a lot more work for us (there's a large backlog of pull requests that
we're making good progress in reviewing and merging).
Sometimes there are very good reasons to split up a PR. For example, the
recent TorrentBroadcast improvements are being performed as a series of
multiple PRs, but I that's okay because those PRs are much larger units of
functionality (e.g. hundreds of lines, instead of < 10) which we might want to
merge and backport separately.
---
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]
