Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19452#discussion_r144435539
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinExec.scala
---
@@ -221,43 +237,36 @@ case class Streaming
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/19467
Whoops, i missed one comment. Not LGTM. Need tests in PlannerSuite that
tests whether EnsureRequirements respects numPartitions in
ClusteredDistribution.
---
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144403807
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/EnsureRequirements.scala
---
@@ -50,7 +50,8 @@ case class EnsureRequirements(conf: S
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144434463
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/StatefulOperatorTest.scala
---
@@ -0,0 +1,49 @@
+/*
+ * Licensed to the Apache Softwa
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144434390
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/StatefulOperatorTest.scala
---
@@ -0,0 +1,49 @@
+/*
+ * Licensed to the Apache Softwa
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144433715
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/StateStoreRDDSuite.scala
---
@@ -214,7 +214,7 @@ class StateStoreRDDSuite ext
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144402006
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/physical/partitioning.scala
---
@@ -243,8 +246,9 @@ case class HashPartitioning(expr
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144408406
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/EnsureStatefulOpPartitioningSuite.scala
---
@@ -53,7 +53,7 @@ class EnsureStatefulOpPartition
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144406045
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -43,10 +43,11 @@ case class StatefulOperatorStateInf
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19467#discussion_r144406113
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -239,7 +240,7 @@ case class StateStoreRestoreExec(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82705/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82705/testReport)**
for PR 19467 at commit
[`5122117`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18029
**[Test build #82708 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82708/testReport)**
for PR 18029 at commit
[`ac0ae53`](https://github.com/apache/spark/commit/ac
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82704/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82704 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82704/testReport)**
for PR 19467 at commit
[`407d76c`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19451
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82703/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19451
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19451
**[Test build #82703 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82703/testReport)**
for PR 19451 at commit
[`24b6b6c`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82702/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82702 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82702/testReport)**
for PR 19448 at commit
[`f486263`](https://github.com/apache/spark/commit/f
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19270
> from my understanding SPARK-20657 is proposing to not using the rest api
Not really; I just don't know enough JS to actually make the change to use
the REST api, but I do believe using it i
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/19390
@susanxhuynh could you review pls?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19270
Right so from my understanding
https://issues.apache.org/jira/browse/SPARK-20657 is proposing to not using the
rest api and force more on the backing store which exists only for history
server bu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19486
**[Test build #82707 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82707/testReport)**
for PR 19486 at commit
[`7c321fc`](https://github.com/apache/spark/commit/7c
GitHub user ash211 opened a pull request:
https://github.com/apache/spark/pull/19486
[SPARK-22268][BUILD] Fix lint-java
## What changes were proposed in this pull request?
Fix java style issues
## How was this patch tested?
Run `./dev/lint-java` locally sin
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19452
**[Test build #82706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82706/testReport)**
for PR 19452 at commit
[`347b92d`](https://github.com/apache/spark/commit/34
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82705/testReport)**
for PR 19467 at commit
[`5122117`](https://github.com/apache/spark/commit/51
Github user jomach commented on the issue:
https://github.com/apache/spark/pull/19485
Yes I will do it. give me some days please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19470
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82701/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19470
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82700/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19470
**[Test build #82701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82701/testReport)**
for PR 19470 at commit
[`8ac1acf`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82700/testReport)**
for PR 19448 at commit
[`42afccb`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19478
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19478
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82698/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19478
**[Test build #82698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82698/testReport)**
for PR 19478 at commit
[`03e88b0`](https://github.com/apache/spark/commit/0
Github user sathiyapk commented on the issue:
https://github.com/apache/spark/pull/19451
@gengliangwang thanks!
I'll squash my commits into 2 if everything is fine..
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19464
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19464
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82696/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19464
**[Test build #82696 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82696/testReport)**
for PR 19464 at commit
[`527b367`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82704/testReport)**
for PR 19467 at commit
[`407d76c`](https://github.com/apache/spark/commit/40
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/19451
LGTM.
@gatorsmile please help check again.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19451
**[Test build #82703 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82703/testReport)**
for PR 19451 at commit
[`24b6b6c`](https://github.com/apache/spark/commit/24
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82702 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82702/testReport)**
for PR 19448 at commit
[`f486263`](https://github.com/apache/spark/commit/f4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19480
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82697/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82699/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82699 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82699/testReport)**
for PR 19467 at commit
[`3f51c5c`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19480
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19480
**[Test build #82697 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82697/testReport)**
for PR 19480 at commit
[`76b5489`](https://github.com/apache/spark/commit/7
Github user joseph-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/19452#discussion_r144396781
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinHelper.scala
---
@@ -66,6 +67,60 @@ object Strea
Github user gengliangwang commented on the issue:
https://github.com/apache/spark/pull/19478
@viirya @hvanhovell @gatorsmile
Thanks, I have attached the performance result in the description in this
PR.
Overall I don't see any downside of the code change.
Also it is p
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18972
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82695/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18972
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18972
**[Test build #82695 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82695/testReport)**
for PR 18972 at commit
[`c9bc395`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19270
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82693/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19270
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19270
**[Test build #82693 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82693/testReport)**
for PR 19270 at commit
[`ef90c2a`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19464
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19464
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82694/
Test FAILed.
---
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144388430
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFileForma
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/19448
Still +1 from me as well.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19464
**[Test build #82694 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82694/testReport)**
for PR 19464 at commit
[`4dcfd83`](https://github.com/apache/spark/commit/4
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/18460
Thanks you, @gatorsmile . Now, it's simplified more.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19470
**[Test build #82701 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82701/testReport)**
for PR 19470 at commit
[`8ac1acf`](https://github.com/apache/spark/commit/8a
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19470#discussion_r144387023
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -2050,4 +2050,80 @@ class SQLQuerySuite extends Quer
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19478
Compare the total optimization time for the TPC-DS queries?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82692/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19448
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82692 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82692/testReport)**
for PR 19448 at commit
[`c93eb1b`](https://github.com/apache/spark/commit/c
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144381367
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19448
**[Test build #82700 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82700/testReport)**
for PR 19448 at commit
[`42afccb`](https://github.com/apache/spark/commit/42
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/19448
LGTM pending Jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144379591
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,10 @@ class ParquetFileF
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18747
@cloud-fan could you please review this since I resolved a confclit?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19470
Hi, @gatorsmile .
- **For mismatched column orders**:
[SPARK-22267](https://issues.apache.org/jira/browse/SPARK-22267) shows the
current status of ORC.
- **For mismatched column types*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18747
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18747
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82691/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18747
**[Test build #82691 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82691/testReport)**
for PR 18747 at commit
[`8134e73`](https://github.com/apache/spark/commit/8
Github user ssaavedra commented on the issue:
https://github.com/apache/spark/pull/19427
Should I create the appropriate issue in JIRA? I'm not sure if there is any
automation which does that.
---
-
To unsubscribe,
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/19448#discussion_r144375059
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -138,6 +138,11 @@ class ParquetFi
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19470
To be clear, I'll file another JIRA about ORC status on mismatched column
orders.
---
-
To unsubscribe, e-mail: reviews-un
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19452#discussion_r144372504
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinExec.scala
---
@@ -387,9 +400,10 @@ case class StreamingS
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19452#discussion_r144371994
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinHelper.scala
---
@@ -66,6 +67,60 @@ object StreamingSymme
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19478
**[Test build #82698 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82698/testReport)**
for PR 19478 at commit
[`03e88b0`](https://github.com/apache/spark/commit/03
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82699 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82699/testReport)**
for PR 19467 at commit
[`3f51c5c`](https://github.com/apache/spark/commit/3f
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19270
The main change I'm talking about is
https://issues.apache.org/jira/browse/SPARK-20657 (code at
https://github.com/vanzin/spark/pull/41). I did not change the format of the
tables, but how data is l
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/19270
@vanzin I haven't followed the prs from SPARK-18085 closely, you keep
mentioning it here but don't give anymore context as to what that has to do
with this change. Did you change the format of ta
Github user michaelmior commented on the issue:
https://github.com/apache/spark/pull/19263
@vanzin Just wanted to ping since I believe I addressed your concerns. Let
me know if it looks like any other changes are required.
---
Github user MrBago commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r144365856
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,256 @@
+/*
+ * Licensed to the Apache Software Foundation (
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144365104
--- Diff: core/src/test/scala/org/apache/spark/FileSuite.scala ---
@@ -510,4 +510,83 @@ class FileSuite extends SparkFunSuite with
LocalSparkContext {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82689/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82690/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19467
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19467
**[Test build #82690 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82690/testReport)**
for PR 19467 at commit
[`48d1f25`](https://github.com/apache/spark/commit/4
201 - 300 of 604 matches
Mail list logo
