Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8678#issuecomment-139339394
I just think having a match evaluated in a method parameter makes the code
look more complicated. Elsewhere in Spark we try to simplify the existing code
in the opposite direction, so I see this as some sort of regression.
If it's actually a significant source of performance improvement then I
would be OK with sacrificing readability, but that's not the case here.
---
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]
