Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/9655#issuecomment-156087424
I agree with you. However, if you see the codes, they are used in a mixed
way, even in the same class.
Mostly the usages are `_ and _` or ` + ` but `xxx` and `xxxLeft` are used
in a mixed way even in the same class. Namely it looks this does not infer that
the order is a matter.
If the order is apparently not a matter, then I think we better set all
`xxx` instead of `xxxLeft`.
Anyhow, I would like to close if guys think this is inappropriate.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
