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