Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/4905#issuecomment-82672776
I'm not really sure I would say these debugging messages are in the "wrong
place". They are in the place that they needed to be when I was debugging this
code, and they aren't really intended for anything other than that. The change
to use .output instead is more concise, but is a no-op. While its good to have
simple code, there is a limited amount of time that can be spent reviewing and
merging PRs and I'd rather most PRs came with test cases that were failing
previously and are now fixed.
---
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]
