{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/10653","labels_url":"https://api.github.com/repos/apache/spark/issues/10653/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/10653/comments","events_url":"https://api.github.com/repos/apache/spark/issues/10653/events","html_url":"https://github.com/apache/spark/pull/10653","id":125506903,"number":10653,"title":"[SPARK-12700] [SQL] embed condition into SMJ and BroadcastHashJoin","user":{"login":"davies","id":40902,"avatar_url":"https://avatars.githubusercontent.com/u/40902?v=3","gravatar_id":"","url":"https://api.github.com/users/davies","html_url":"https://github.com/davies","followers_url":"https://api.github.com/users/davies/followers","following_url":"https://api.github.com/users/davies/following{/other_user}","gists_url":"https://api.github.com/users/davies/gists{/gist_id}","starred_url":"https://api.github.com/users/davies/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davies/subscriptions","organizations_url":"https://api.github.com/users/davies/orgs","repos_url":"https://api.github.com/users/davies/repos","events_url":"https://api.github.com/users/davies/events{/privacy}","received_events_url":"https://api.github.com/users/davies/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-01-07T23:16:04Z","updated_at":"2016-01-07T23:45:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/10653","html_url":"https://github.com/apache/spark/pull/10653","diff_url":"https://github.com/apache/spark/pull/10653.diff","patch_url":"https://github.com/apache/spark/pull/10653.patch"},"body":"Currently SortMergeJoin and BroadcastHashJoin do not support condition, the need a followed Filter for that, the result projection to generate UnsafeRow could be very expensive if they generate lots of rows and could be filtered mostly by condition.\r\n\r\nThis PR brings the support of condition for SortMergeJoin and BroadcastHashJoin, just like other outer joins do.\r\n\r\nThis could improve the performance of Q72 by 7x (from 120s to 16.5s)."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/169842586","html_url":"https://github.com/apache/spark/pull/10653#issuecomment-169842586","issue_url":"https://api.github.com/repos/apache/spark/issues/10653","id":169842586,"user":{"login":"AmplabJenkins","id":4031291,"avatar_url":"https://avatars.githubusercontent.com/u/4031291?v=3","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2016-01-07T23:45:47Z","updated_at":"2016-01-07T23:45:47Z","body":"Merged build finished. Test FAILed."}}