Github user jacek-lewandowski commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121929208
@JoshRosen this shouldn't be a problem for us. Thanks for pinging me anyway.
---
If your project is set up for it, you can reply to this email and have your
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34746914
--- Diff: core/src/main/scala/org/apache/spark/rpc/akka/AkkaRpcEnv.scala ---
@@ -237,6 +237,12 @@ private[spark] class AkkaRpcEnv private[akka] (
}
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34727955
--- Diff: core/src/main/scala/org/apache/spark/rpc/akka/AkkaRpcEnv.scala ---
@@ -237,6 +237,12 @@ private[spark] class AkkaRpcEnv private[akka] (
}
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121746386
Thanks - I've merged 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 does not have
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7159
---
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 andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121744779
Hi @zsxwing this LGTM. Feel free to merge 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121385885
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121385911
Merged build started.
---
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
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34622347
--- Diff:
core/src/main/scala/org/apache/spark/deploy/master/ZooKeeperPersistenceEngine.scala
---
@@ -57,17 +58,16 @@ private[master] class
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121402767
I left one question, but LGTM otherwise.
---
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/7159#issuecomment-121386123
[Test build #37265 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37265/consoleFull)
for PR 7159 at commit
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34623467
--- Diff: core/src/main/scala/org/apache/spark/rpc/akka/AkkaRpcEnv.scala ---
@@ -237,6 +237,12 @@ private[spark] class AkkaRpcEnv private[akka] (
}
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34623503
--- Diff: core/src/main/scala/org/apache/spark/rpc/akka/AkkaRpcEnv.scala ---
@@ -237,6 +237,12 @@ private[spark] class AkkaRpcEnv private[akka] (
}
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121384689
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 andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34622175
--- Diff: core/src/main/scala/org/apache/spark/rpc/akka/AkkaRpcEnv.scala ---
@@ -237,6 +237,12 @@ private[spark] class AkkaRpcEnv private[akka] (
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121440155
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121440172
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121443550
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 project does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121438940
Build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121438881
Build triggered.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121441428
[Test build #37292 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37292/consoleFull)
for PR 7159 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121471289
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/7159#issuecomment-121471252
[Test build #37292 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37292/console)
for PR 7159 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121428386
[Test build #37265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37265/console)
for PR 7159 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121428434
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121430166
/ping @jlewandowski, just want to give you a head's up about the
binary-incompatible change to RecoveryModeFactory. I don't think that we can
avoid this change, but
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r34636951
--- Diff: core/src/main/scala/org/apache/spark/rpc/akka/AkkaRpcEnv.scala ---
@@ -237,6 +237,12 @@ private[spark] class AkkaRpcEnv private[akka] (
}
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-120908482
ping @andrewor14
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-121071123
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118361204
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118361215
Merged build started.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118361420
[Test build #36502 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36502/consoleFull)
for PR 7159 at commit
Github user adamv commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r33875006
--- Diff: core/pom.xml ---
@@ -373,6 +373,10 @@
scopetest/scope
/dependency
dependency
+ groupIdorg.apache.curator/groupId
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118393327
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/7159#issuecomment-118393293
[Test build #36502 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36502/console)
for PR 7159 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r33883790
--- Diff: core/pom.xml ---
@@ -373,6 +373,10 @@
scopetest/scope
/dependency
dependency
+
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118449442
[Test build #36519 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36519/consoleFull)
for PR 7159 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118449361
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118449363
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118459399
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/7159#issuecomment-118459389
[Test build #36519 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36519/console)
for PR 7159 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118040584
[Test build #36398 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36398/console)
for PR 7159 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118043747
[Test build #36400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36400/consoleFull)
for PR 7159 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/7159#discussion_r33778479
--- Diff: core/src/main/scala/org/apache/spark/rpc/RpcEnv.scala ---
@@ -138,6 +138,12 @@ private[spark] abstract class RpcEnv(conf: SparkConf) {
*
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118037990
[Test build #36398 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36398/consoleFull)
for PR 7159 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118040634
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/7159#issuecomment-118043319
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118043278
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118037519
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118037497
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118077075
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/7159#issuecomment-118079688
[Test build #36408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36408/consoleFull)
for PR 7159 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118099784
cc @andrewor14 can you review 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118114688
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/7159#issuecomment-118114516
[Test build #36408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36408/console)
for PR 7159 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118077035
[Test build #36400 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36400/console)
for PR 7159 at commit
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118077915
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118078871
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-118078916
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117737815
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/7159#issuecomment-117690470
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117690442
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117683031
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117683052
Merged build started.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117683424
[Test build #36264 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36264/consoleFull)
for PR 7159 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117691091
[Test build #36265 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36265/consoleFull)
for PR 7159 at commit
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117682602
/cc @rxin
---
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 opened a pull request:
https://github.com/apache/spark/pull/7159
[SPARK-6602][Core]Replace Akka Serialization with Spark Serializer
Replace Akka Serialization with Spark Serializer and add unit tests.
You can merge this pull request into a Git repository by
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117725169
This is the last PR for SPARK-6602
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7159#issuecomment-117737501
[Test build #36265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36265/console)
for PR 7159 at commit
70 matches
Mail list logo