Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-153116416
Merging this to master.
---
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 zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-153119171
Just took another look. 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9210
---
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/9210#issuecomment-152887248
**[Test build #44778 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44778/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-152887283
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/9210#issuecomment-152908550
**[Test build #44787 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44787/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-152908584
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/9210#issuecomment-152908585
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/9210#issuecomment-152887620
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/9210#issuecomment-152887634
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-152887334
hmm, akka test... it's failed for me in unrelated changes, also. retest
this please.
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-152887284
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/9210#issuecomment-152888646
**[Test build #44787 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44787/consoleFull)**
for PR 9210 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-152873572
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/9210#issuecomment-152875833
**[Test build #44778 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44778/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-152874457
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/9210#issuecomment-152874465
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-151760604
hi there, any remaining feedback here?
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150846405
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/9210#issuecomment-150854175
**[Test build #44298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44298/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150854195
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/9210#issuecomment-150854194
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/9210#issuecomment-150846606
**[Test build #44298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44298/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150496646
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/9210#issuecomment-150496548
**[Test build #44200 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44200/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150496647
Test PASSed.
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/9210#issuecomment-150500153
**[Test build #44199 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44199/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150500252
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 project does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150500253
Test PASSed.
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/9210#issuecomment-150128910
**[Test build #44127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44127/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150129053
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/9210#issuecomment-150129051
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/9210#issuecomment-150378799
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/9210#issuecomment-150378777
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/9210#issuecomment-150396160
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/9210#issuecomment-150396161
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/9210#issuecomment-150396121
**[Test build #44185 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44185/consoleFull)**
for PR 9210 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150379910
**[Test build #44185 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44185/consoleFull)**
for PR 9210 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150396478
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/9210#issuecomment-150396653
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/9210#issuecomment-150396638
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150397497
**[Test build #44187 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44187/consoleFull)**
for PR 9210 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42756277
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -294,47 +319,68 @@ private[netty] class NettyRpcEnvFactory extends
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42830849
--- Diff: core/src/main/scala/org/apache/spark/SparkEnv.scala ---
@@ -266,7 +267,7 @@ object SparkEnv extends Logging {
// Figure out which port Akka
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42832803
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -184,8 +184,14 @@ private[netty] class NettyRpcEnv(
//
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150420011
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/9210#issuecomment-150420007
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 rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42830874
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -89,8 +90,7 @@ private[netty] class NettyRpcEnv(
}
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42830903
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -184,8 +184,14 @@ private[netty] class NettyRpcEnv(
//
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150468709
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/9210#issuecomment-150468698
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/9210#issuecomment-150469255
**[Test build #44199 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44199/consoleFull)**
for PR 9210 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150419454
**[Test build #44187 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44187/consoleFull)**
for PR 9210 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42830999
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -399,41 +445,41 @@ private[netty] case class Ack(sender:
NettyRpcEndpointRef)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150477583
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150477946
**[Test build #44200 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44200/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150477601
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 zsxwing commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150254402
Just a nit. Otherwise 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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150105131
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/9210#issuecomment-150105203
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/9210#issuecomment-150105216
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/9210#issuecomment-150106021
**[Test build #44127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44127/consoleFull)**
for PR 9210 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42700940
--- Diff: network/yarn/pom.xml ---
@@ -54,6 +54,11 @@
org.apache.hadoop
hadoop-client
+
--- End diff --
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42701027
--- Diff:
core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala
---
@@ -80,9 +78,8 @@ private[spark] class
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42701057
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -223,9 +229,11 @@ private[netty] class NettyRpcEnv(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150065308
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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9210#discussion_r42701122
--- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcEnv.scala
---
@@ -399,41 +445,41 @@ private[netty] case class Ack(sender:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150065294
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150066237
**[Test build #44110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44110/consoleFull)**
for PR 9210 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9210#issuecomment-150097552
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/9210#issuecomment-150097554
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/9210#issuecomment-150097532
**[Test build #44110 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44110/consoleFull)**
for PR 9210 at commit
72 matches
Mail list logo