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
[`c6d2a15`](https://gi
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
[`cffbc13`](https://gi
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 projec
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 do
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 projec
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
[`90a1432`](https://g
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):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
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 SparkFunSuit
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
[`90a1432`](https://gi
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 {
*/
def
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 SparkFunSuit
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
[`2103175`](https://g
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):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
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
[`2103175`](https://gi
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 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 new
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 ne
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
[`3754218`](https://gi
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 new
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 projec
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):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
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
[`f8eb88e`](https://g
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 us
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 ne
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 us
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
[`f8eb88e`](https://gi
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
N
27 matches
Mail list logo
