Github user zsxwing closed the pull request at:
https://github.com/apache/spark/pull/10656
---
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
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/10656#issuecomment-169951385
Merging this! Thanks!
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169833544
**[Test build #48979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48979/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169855045
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169855043
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169854898
**[Test build #48979 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48979/consoleFull)**
for PR 10609 at commit
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169840672
By the way, I will send another PR for branch 1.6 due to the conflicts of
MimaExcludes.scala.
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169843683
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169843686
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169843942
retest this please
---
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
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/10656
[SPARK-12591][Streaming]Register OpenHashMapBasedStateMap for Kryo (branch
1.6)
backport #10609 to branch 1.6
You can merge this pull request into a Git repository by running:
$ git pull
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10656#issuecomment-169881498
**[Test build #48997 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48997/consoleFull)**
for PR 10656 at commit
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169863068
LGTM. Merging this to master. Please send another PR for 1.6 ASAP.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10609
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10656#issuecomment-169897645
**[Test build #48997 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48997/consoleFull)**
for PR 10656 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10656#issuecomment-169897704
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10656#issuecomment-169897706
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10656#issuecomment-169899417
CC @tdas
---
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
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49122591
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StateMapSuite.scala ---
@@ -321,4 +336,59 @@ class StateMapSuite extends SparkFunSuite {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169788713
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169788654
**[Test build #48960 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48960/consoleFull)**
for PR 10609 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169793287
**[Test build #48963 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48963/consoleFull)**
for PR 10609 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49121790
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -408,9 +409,11 @@ private[serializer] class
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49122800
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StateMapSuite.scala ---
@@ -321,4 +336,59 @@ class StateMapSuite extends SparkFunSuite {
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49111736
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/StateMap.scala ---
@@ -77,6 +80,7 @@ private[streaming] class EmptyStateMap[K: ClassTag,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169780118
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169785460
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49112106
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/StateMap.scala ---
@@ -77,21 +81,26 @@ private[streaming] class EmptyStateMap[K:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169781200
**[Test build #48958 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48958/consoleFull)**
for PR 10609 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169784539
**[Test build #48960 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48960/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169817531
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169817535
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169818726
retest this please
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169820507
**[Test build #48963 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48963/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169820761
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169820758
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169822732
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169822736
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169803306
LGTM!
---
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
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49127069
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -426,6 +439,7 @@ private[serializer] class
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169811230
retest this please
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169803074
**[Test build #48967 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48967/consoleFull)**
for PR 10609 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49125114
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -426,6 +439,7 @@ private[serializer] class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169809439
**[Test build #48969 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48969/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169809796
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169809785
**[Test build #48969 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48969/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169809797
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169828020
**[Test build #48967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48967/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169828271
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169828273
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169831500
retest this.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169270419
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169270414
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169270236
**[Test build #48843 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48843/consoleFull)**
for PR 10609 at commit
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169412353
CC @tdas
---
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
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r48993279
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -166,6 +166,15 @@ class KryoSerializer(conf: SparkConf)
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r48993619
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -166,6 +166,15 @@ class KryoSerializer(conf: SparkConf)
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169508519
**[Test build #48884 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48884/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169516182
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169516181
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 user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49034101
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StateMapSuite.scala ---
@@ -267,12 +270,17 @@ class StateMapSuite extends SparkFunSuite {
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169516066
**[Test build #48884 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48884/consoleFull)**
for PR 10609 at commit
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169519483
@tdas updated as we discussed offline.
---
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 user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/10609#discussion_r49034162
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/StateMap.scala ---
@@ -77,6 +80,7 @@ private[streaming] class EmptyStateMap[K: ClassTag, S:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169253775
**[Test build #48843 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48843/consoleFull)**
for PR 10609 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169210384
**[Test build #48808 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48808/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169210522
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169210523
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169191880
**[Test build #48798 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48798/consoleFull)**
for PR 10609 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169192001
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169191999
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169175648
**[Test build #48798 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48798/consoleFull)**
for PR 10609 at commit
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/10609
[SPARK-12591][Streaming]Register OpenHashMapBasedStateMap for Kryo
The default serializer in Kryo is FieldSerializer and it ignores transient
fields and never calls `writeObject` or `readObject`.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10609#issuecomment-169196126
**[Test build #48808 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48808/consoleFull)**
for PR 10609 at commit
74 matches
Mail list logo