Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10630#issuecomment-176630203
This is not that big. Let's just do it together 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 y
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176629923
@rxin @liancheng yes, I would love to do it. 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 y
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176629661
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176629658
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/10757#issuecomment-176629629
**[Test build #50356 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50356/consoleFull)**
for PR 10757 at commit
[`eef9423`](https://g
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9657#discussion_r51233700
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ResolvedDataSource.scala
---
@@ -235,11 +236,18 @@ object ResolvedDataSource exte
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10930#issuecomment-176629169
**[Test build #50357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50357/consoleFull)**
for PR 10930 at commit
[`c5e4d99`](https://gi
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10630#issuecomment-176628957
LGTM. we can merge it first and @gatorsmile can address remaining comments
in a follow-up PR.
---
If your project is set up for it, you can reply to this email and h
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10630#discussion_r51233589
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisErrorSuite.scala
---
@@ -270,7 +270,8 @@ class AnalysisErrorSuite ext
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10677#issuecomment-176628618
@hvanhovell can you review this one?
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10630#discussion_r51233504
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceOperatorSuite.scala
---
@@ -0,0 +1,46 @@
+/*
+ * Licensed to
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9657#discussion_r51233418
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/WriterContainer.scala
---
@@ -239,8 +249,10 @@ private[sql] abstract class BaseWr
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10630#discussion_r51233323
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala
---
@@ -315,4 +315,7 @@ abstract class BinaryNode exte
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/10961#discussion_r51233270
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVectorUtils.java
---
@@ -59,19 +62,44 @@ public static Object
toPrimitiveJa
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9657#discussion_r51233196
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/ResolvedDataSource.scala
---
@@ -235,11 +236,18 @@ object ResolvedDataSource exte
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/10961#discussion_r51233211
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVectorUtils.java
---
@@ -59,19 +62,44 @@ public static Object
toPrimitiveJa
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10630#discussion_r51233135
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -214,12 +214,22 @@ trait CheckAnalysis {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10630#discussion_r51233162
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -214,12 +214,22 @@ trait CheckAnalysis {
Github user liancheng closed the pull request at:
https://github.com/apache/spark/pull/10972
---
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 liancheng commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176622130
I'm closing this one.
---
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 th
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10954
---
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 ena
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/10961#discussion_r51232913
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVector.java
---
@@ -354,6 +435,33 @@ public void reset() {
/**
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/10961#discussion_r51232873
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVector.java
---
@@ -282,6 +328,21 @@ public void reset() {
/**
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9657#discussion_r51232882
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -155,6 +157,15 @@ final class DataFrameWriter private[sql](df:
DataFrame)
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9657#discussion_r51232826
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -155,6 +157,15 @@ final class DataFrameWriter private[sql](df:
DataFrame)
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/9657#discussion_r51232808
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -130,6 +131,16 @@ final class DataFrameWriter private[sql](df:
DataFrame)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176620545
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176620544
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/10977#issuecomment-176620367
**[Test build #50351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50351/consoleFull)**
for PR 10977 at commit
[`951e2cd`](https://g
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10954#issuecomment-176620214
I'm merging this. Thanks for fixing 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 proj
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176620209
**[Test build #50356 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50356/consoleFull)**
for PR 10757 at commit
[`eef9423`](https://gi
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/10961#discussion_r51232647
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVector.java
---
@@ -126,12 +147,25 @@ public ArrayData copy() {
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/10961#discussion_r51232603
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVector.java
---
@@ -126,12 +147,25 @@ public ArrayData copy() {
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176619981
The issue here is that we want test cases that are targeted for specific
problems, and the Hive ones are not (they are just a giant blackbox we took to
bootstrap coverage)
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10954#issuecomment-176619493
@vanzin I see, thanks for the explanation! Then this LGTM.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176619523
Sure it's a good idea to use that golden file infrastructure. Given we
don't have that yet, can you just add a test case?
---
If your project is set up for it, you
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10860#issuecomment-176619085
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/10860#issuecomment-176618998
**[Test build #50349 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50349/consoleFull)**
for PR 10860 at commit
[`772f045`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10860#issuecomment-176619087
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 davies commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176618967
The way we managed HiveCompatibilitySuite is actually better than our unit
tests (sql query and golden results in text format). Even if we don't want to
be compatible wi
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176618727
@viirya It would be great if you can help since you are pretty familiar
with this part of code :)
---
If your project is set up for it, you can reply to this email a
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10961#issuecomment-176618365
cc @davies for a more detailed review
---
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/10972#discussion_r51232373
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRelation.scala
---
@@ -799,9 +799,26 @@ private[sql] object Pa
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176617106
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176617110
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/10972#issuecomment-176616985
**[Test build #50354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50354/consoleFull)**
for PR 10972 at commit
[`2e5eddb`](https://g
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176616783
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 liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/9940#discussion_r51232182
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala ---
@@ -359,6 +361,18 @@ object StructType extends AbstractDataType {
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10974#issuecomment-176613193
@rxin similar to https://github.com/apache/spark/pull/10333 , I only
observed about 5% speed up. I think branch prediction works very well for this
kind of case, the
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176613206
@viirya do you want to submit a pull request to address your issue?
@liancheng is busy with something right now.
---
If your project is set up for it, you can reply to t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10723#issuecomment-176612064
**[Test build #50355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50355/consoleFull)**
for PR 10723 at commit
[`8b7086e`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176609589
**[Test build #50353 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50353/consoleFull)**
for PR 10757 at commit
[`eef9423`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176609786
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176609793
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176609355
Thanks - can you add a test case that would catch this? In the long run, we
don't want to rely on HiveCompatibilitySuite.
---
If your project is set up for it, you
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10978#issuecomment-176609044
Can one of the admins verify this patch?
---
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 p
GitHub user flyjy opened a pull request:
https://github.com/apache/spark/pull/10978
[SPARK-13074][Core] Add JavaSparkContext. getPersistentRDDs method
The "getPersistentRDDs()" is a useful API of SparkContext to get cached
RDDs. However, the JavaSparkContext does not have this API.
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10970#issuecomment-176608171
Yea the pull request description gets merged as commit description, and is
the only thing that sticks around forever.
---
If your project is set up for it, you can reply
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10757#discussion_r51231438
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/namedExpressions.scala
---
@@ -298,7 +299,14 @@ case class AttributeReference(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10973#issuecomment-176607805
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10973#issuecomment-176607807
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/10973#issuecomment-176607734
**[Test build #50348 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50348/consoleFull)**
for PR 10973 at commit
[`c5a97fc`](https://g
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/10970#issuecomment-176607697
Sorry I thought the JIRA description tells it, will add it to PR
description next time.
---
If your project is set up for it, you can reply to this email and have yo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10972#issuecomment-176607674
**[Test build #50354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50354/consoleFull)**
for PR 10972 at commit
[`2e5eddb`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176607410
A few comments:
1. As discussed offline, let's be consistent and always quote in sql query
generation, and never allow backticks in aliases.
2. Rename saf
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10970#issuecomment-176605624
OK got it. @cloud-fan Please add that to the pull request description next
time.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/10970#issuecomment-176605185
Existing tests were failing locally, but not on Jenkins. We are not sure
why.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10757#issuecomment-176604129
**[Test build #50353 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50353/consoleFull)**
for PR 10757 at commit
[`eef9423`](https://gi
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10970
---
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 ena
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10916
---
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 ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10970#issuecomment-176603047
@cloud-fan is this fixing a bug? If yes, please add a test somewhere.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/10970#issuecomment-176602785
Tested it locally, test passed. I'm merging this into master, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/10757#discussion_r51230274
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -39,11 +39,11 @@ case class ScalaUDF(
dataT
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10057#issuecomment-176602007
**[Test build #50352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50352/consoleFull)**
for PR 10057 at commit
[`5ebaf04`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176601896
**[Test build #50351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50351/consoleFull)**
for PR 10977 at commit
[`951e2cd`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10916#issuecomment-176601335
**[Test build #50350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50350/consoleFull)**
for PR 10916 at commit
[`ff5508b`](https://gi
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176601291
When there is no aggregate functions, it did not generate the output using
resultExpression, which have only literals (I was mislead by the comment in
AggregateIterator)
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10977#issuecomment-176600324
What's the bug?
---
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 davies opened a pull request:
https://github.com/apache/spark/pull/10977
[SPARK-13031] [SQL] cleanup codegen and improve test coverage
1. enable whole stage codegen during tests even there is only one operator
supports that.
2. split doProduce() into two APIs: upstre
Github user tnachen commented on the pull request:
https://github.com/apache/spark/pull/10057#issuecomment-176599484
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 fe
Github user tnachen commented on the pull request:
https://github.com/apache/spark/pull/8872#issuecomment-176599538
Looks like there are more rules to scala style now, it's finally passing!
@andrewor14 PTAL
---
If your project is set up for it, you can reply to this email and have y
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10964#issuecomment-176599392
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10964#issuecomment-176599391
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10964#issuecomment-176599145
**[Test build #50339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50339/consoleFull)**
for PR 10964 at commit
[`2244999`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10974#issuecomment-176599074
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10974#issuecomment-176599067
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/9893#issuecomment-176598841
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10916#issuecomment-176598666
Thanks - I've merged 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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9893#issuecomment-176598833
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10916#issuecomment-176598689
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10916#issuecomment-176598683
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10974#issuecomment-176598205
**[Test build #50341 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50341/consoleFull)**
for PR 10974 at commit
[`f6a3ccc`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9893#issuecomment-176598247
**[Test build #50340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50340/consoleFull)**
for PR 9893 at commit
[`33fd2c4`](https://git
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/10916#issuecomment-176598233
@rxin I've reverted last commit.
---
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/10916#issuecomment-176597875
**[Test build #50337 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50337/consoleFull)**
for PR 10916 at commit
[`87c27ef`](https://g
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/10860#discussion_r51229062
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -248,9 +248,7 @@ private[history] class FsHistoryProvider(co
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10860#issuecomment-176591953
**[Test build #50349 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50349/consoleFull)**
for PR 10860 at commit
[`772f045`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9657#issuecomment-176589925
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10976#issuecomment-176590073
Can one of the admins verify this patch?
---
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 p
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9657#issuecomment-176589924
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 project
1 - 100 of 627 matches
Mail list logo