Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15905
I personally think it's good to be consistent. I think it's more readable
to break out these function definitions, and, it seems like there's evidence it
might avoid some unintended objects in a closure. Have a look for other
instances of "seqOp = ..." etc and see which ones look like the same pattern
that could be refactored.
---
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]
