{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/7940","labels_url":"https://api.github.com/repos/apache/spark/issues/7940/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/7940/comments","events_url":"https://api.github.com/repos/apache/spark/issues/7940/events","html_url":"https://github.com/apache/spark/pull/7940","id":99037555,"number":7940,"title":"[SPARK-9482] [SQL] Fix race condition in BroadcastHashJoin/BroadcastHashOuterJoin","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":24,"created_at":"2015-08-04T18:58:30Z","updated_at":"2015-08-04T23:33:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/7940","html_url":"https://github.com/apache/spark/pull/7940","diff_url":"https://github.com/apache/spark/pull/7940.diff","patch_url":"https://github.com/apache/spark/pull/7940.patch"},"body":"BroadcastHashJoin/BroadcastHashOuterJoin could be created (or copied) multiple times during catalyst planner, we should not create the broadcast in the constructor of them, so just do them in doExecutor or using `lazy val` to reuse the broadcast.\r\n\r\nThis PR also change to use `def` instead of `lazy val` for UnsafeProjection, because it's not thread safe.\r\n\r\nTODO: cleanup the debug code once the flaky test passed 100 times."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/127796519","html_url":"https://github.com/apache/spark/pull/7940#issuecomment-127796519","issue_url":"https://api.github.com/repos/apache/spark/issues/7940","id":127796519,"user":{"login":"SparkQA","id":8108735,"avatar_url":"https://avatars.githubusercontent.com/u/8108735?v=3","gravatar_id":"","url":"https://api.github.com/users/SparkQA","html_url":"https://github.com/SparkQA","followers_url":"https://api.github.com/users/SparkQA/followers","following_url":"https://api.github.com/users/SparkQA/following{/other_user}","gists_url":"https://api.github.com/users/SparkQA/gists{/gist_id}","starred_url":"https://api.github.com/users/SparkQA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SparkQA/subscriptions","organizations_url":"https://api.github.com/users/SparkQA/orgs","repos_url":"https://api.github.com/users/SparkQA/repos","events_url":"https://api.github.com/users/SparkQA/events{/privacy}","received_events_url":"https://api.github.com/users/SparkQA/received_events","type":"User","site_admin":false},"created_at":"2015-08-04T23:33:50Z","updated_at":"2015-08-04T23:33:50Z","body":"  [Test build #39753 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39753/console) for   PR 7940 at commit [`0fdacaf`](https://github.com/apache/spark/commit/0fdacaf17bba12f21ffd2ee13fcb71403f30e838).\n * This patch **fails Spark unit tests**.\n * This patch merges cleanly.\n * This patch adds no public classes."}}