Github user piaozhexiu closed the pull request at:
https://github.com/apache/spark/pull/8512
---
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 piaozhexiu commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-207493828
@srowen sure, done!
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-207380286
@piaozhexiu close this in favor of
https://github.com/apache/spark/pull/11242 it seems?
---
If your project is set up for it, you can reply to this email and have your
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r47081031
--- Diff: core/pom.xml ---
@@ -40,6 +40,16 @@
${avro.mapred.classifier}
+ com.amazonaws
+
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r47081146
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkS3UtilSuite.scala ---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to the Apache Software
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r47081737
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,342 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r46992201
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r46991325
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r46992376
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user steveloughran commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-162974033
Has anyone looked at the performance of this versus S3a in Hadoop 2.7+?
Because while I do agree this will dramatically improve s3n: and s3: perf, all
ongoing
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-162996871
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/8512#issuecomment-162996870
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/8512#issuecomment-162996514
**[Test build #47360 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47360/consoleFull)**
for PR 8512 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-162996866
**[Test build #47360 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47360/consoleFull)**
for PR 8512 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-163002146
**[Test build #47362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47362/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-163065488
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/8512#issuecomment-163065410
**[Test build #47362 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47362/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-163065486
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/8512#issuecomment-163067717
**[Test build #47377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47377/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-163085307
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/8512#issuecomment-163085306
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/8512#issuecomment-163085217
**[Test build #47377 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47377/consoleFull)**
for PR 8512 at commit
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r44330648
--- Diff: core/pom.xml ---
@@ -40,6 +40,11 @@
${avro.mapred.classifier}
+ com.amazonaws
+ aws-java-sdk
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154915394
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/8512#issuecomment-154915385
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/8512#issuecomment-154917592
[Test build #45338 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45338/consoleFull)
for PR 8512 at commit
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r44240260
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +66,23 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154915335
[Test build #45337 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45337/consoleFull)
for PR 8512 at commit
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r44240487
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154972016
[Test build #45337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45337/console)
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154972535
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/8512#issuecomment-154984359
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154984021
[Test build #45338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45338/console)
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154915021
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-154915032
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 piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r44240249
--- Diff: core/pom.xml ---
@@ -40,6 +40,11 @@
${avro.mapred.classifier}
+ com.amazonaws
+ aws-java-sdk
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43717724
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43717704
--- Diff: core/pom.xml ---
@@ -40,6 +40,11 @@
${avro.mapred.classifier}
+ com.amazonaws
+ aws-java-sdk
+
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43576793
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +66,23 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43576761
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +66,23 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43576785
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +66,23 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152777962
@pwendell If you are good with the added dependency to core, I will merge
it.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152657561
test 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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152657574
test 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 yhuai commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152657569
test 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/8512#issuecomment-152675362
**[Test build #44706 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44706/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152675441
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/8512#issuecomment-152657162
**[Test build #44701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44701/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152657246
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/8512#issuecomment-152657247
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/8512#issuecomment-152675440
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 piaozhexiu commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152688590
@yhuai Thank you for reviewing! I think I addressed all your comments. Let
me know if you have any further comments.
---
If your project is set up for it, you can
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43567327
--- Diff: core/pom.xml ---
@@ -40,6 +40,11 @@
${avro.mapred.classifier}
+ com.amazonaws
+ aws-java-sdk
+
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152659191
**[Test build #44706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44706/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152624053
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/8512#issuecomment-152623983
**[Test build #44691 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44691/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152624055
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43530868
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +64,20 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152599055
**[Test build #44691 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44691/consoleFull)**
for PR 8512 at commit
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43530739
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43530758
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152598139
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 piaozhexiu commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43530971
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +64,20 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152598097
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/8512#issuecomment-152657775
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/8512#issuecomment-152657805
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 piaozhexiu commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152635891
The test failures (`FlumeStreamSuite`) seem unrelated. I'll rebase to force
another run of build.
---
If your project is set up for it, you can reply to this email
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152636209
test 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/8512#issuecomment-152636758
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/8512#issuecomment-152636725
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/8512#issuecomment-152637655
**[Test build #44701 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44701/consoleFull)**
for PR 8512 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152255040
LGTM, @yhuai Could you take a final look?
---
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/8512#issuecomment-152248073
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/8512#issuecomment-152248104
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/8512#issuecomment-152251272
**[Test build #44612 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44612/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152290834
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/8512#issuecomment-152290672
**[Test build #44612 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44612/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152290833
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 yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43437967
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43439266
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkS3Util.scala ---
@@ -0,0 +1,336 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43439962
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +64,20 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-152310099
@piaozhexiu Thank you for updating it! I left a few comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8512#discussion_r43440241
--- Diff: core/src/main/scala/org/apache/spark/rdd/UnionRDD.scala ---
@@ -62,7 +64,20 @@ class UnionRDD[T: ClassTag](
var rdds: Seq[RDD[T]])
Github user piaozhexiu commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-151907100
@yhuai @liancheng @marmbrus
Sorry to bug you again, but can you review this patch sometime soon?
In fact, I am leaving Netflix on Nov 3rd, and I am
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-151674046
**[Test build #44476 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44476/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-151689877
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/8512#issuecomment-151689879
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/8512#issuecomment-151689805
**[Test build #44476 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44476/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-151673388
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/8512#issuecomment-151673408
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/8512#issuecomment-150748668
**[Test build #44280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44280/consoleFull)**
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-150748702
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/8512#issuecomment-150748701
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/8512#issuecomment-150734465
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/8512#issuecomment-150734581
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/8512#issuecomment-150736390
**[Test build #44280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44280/consoleFull)**
for PR 8512 at commit
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-148841351
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 piaozhexiu commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-148582745
@yhuai @liancheng @davies
I updated the patch incorporating all your comments. Can you please take a
look again? Thanks!
---
If your project is set up for
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-148560037
[Test build #43818 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43818/consoleFull)
for PR 8512 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8512#issuecomment-148559743
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
1 - 100 of 230 matches
Mail list logo
