[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215740221 **[Test build #57342 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57342/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215738820 **[Test build #57340 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57340/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215735912 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215736723 LGTM, can you create a JIRA for it? --- 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

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215737724 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215737708 **[Test build #57339 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57339/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215737730 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61586664 --- Diff: core/src/test/scala/org/apache/spark/InternalAccumulatorSuite.scala --- @@ -188,13 +188,18 @@ class InternalAccumulatorSuite extends

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215737238 **[Test build #57339 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57339/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61586305 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -209,9 +210,9 @@ private[spark] object AccumulatorContext { */

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61586223 --- Diff: core/src/test/scala/org/apache/spark/InternalAccumulatorSuite.scala --- @@ -188,13 +188,18 @@ class InternalAccumulatorSuite extends

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215735899 **[Test build #57338 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57338/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215735915 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215735133 **[Test build #57338 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57338/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61585641 --- Diff: core/src/test/scala/org/apache/spark/InternalAccumulatorSuite.scala --- @@ -188,13 +188,18 @@ class InternalAccumulatorSuite extends SparkFunSuite

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61585127 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -209,9 +210,9 @@ private[spark] object AccumulatorContext { */ def

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61581958 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -209,9 +210,9 @@ private[spark] object AccumulatorContext { */ def

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215709456 **[Test build #57334 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57334/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61576634 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -209,9 +210,9 @@ private[spark] object AccumulatorContext { */ def

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215681965 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215681967 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215681805 **[Test build #57325 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57325/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61551752 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -199,7 +199,8 @@ private[spark] object AccumulatorContext { * TODO: Don't

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61551583 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -209,9 +210,9 @@ private[spark] object AccumulatorContext { */ def

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12776#discussion_r61551567 --- Diff: core/src/main/scala/org/apache/spark/NewAccumulator.scala --- @@ -199,7 +199,8 @@ private[spark] object AccumulatorContext { * TODO: Don't

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12776#issuecomment-215661922 **[Test build #57325 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57325/consoleFull)** for PR 12776 at commit

[GitHub] spark pull request: Use ConcurrentHashMap in place of HashMap for ...

2016-04-29 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/spark/pull/12776 Use ConcurrentHashMap in place of HashMap for NewAccumulator.originals ## What changes were proposed in this pull request? This PR proposes to use ConcurrentHashMap in place of HashMap for