Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10762#issuecomment-171890953 Thanks for making the change -- after looking at the change I think this new version has too many changes (it is fairly ugly that we need to update a lot of files because of the flag). Are there ways to simplify this? When I suggested not introducing a new operator, I was thinking about just changing the join type to something like case class NaturalJoin(tpe: JoinType) extends JoinType
--- 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