Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21052
See my PR https://github.com/apache/spark/pull/21147. We need to fix the
issue first.
---
-
To unsubscribe, e-mail:
Github user mshtelma commented on the issue:
https://github.com/apache/spark/pull/21052
@gatorsmile should I create new PR with these changes for 2.3 branch ? I
will do this. Do we need new jira for 2.3 ? or should I reference the existing
one ?
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21052
@mshtelma Could you submit a backport PR to Spark 2.3?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21052
Let me revert it from Spark 2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/21052
@gatorsmile this broke 2.3 compilation.
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Compile/job/spark-branch-2.3-compile-maven-hadoop-2.6/638/
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21052
Thanks! Merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89684/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89684 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89684/testReport)**
for PR 21052 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89684 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89684/testReport)**
for PR 21052 at commit
Github user mshtelma commented on the issue:
https://github.com/apache/spark/pull/21052
@gatorsmile I have removed explain() and changed formatting
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21052
LGTM except two minor comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89675/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89675 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89675/testReport)**
for PR 21052 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89675 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89675/testReport)**
for PR 21052 at commit
Github user mshtelma commented on the issue:
https://github.com/apache/spark/pull/21052
@maropu thank you for the suggestions! I have implemented them and pushed
the changes.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89656/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89656/testReport)**
for PR 21052 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89656 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89656/testReport)**
for PR 21052 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21052
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mshtelma commented on the issue:
https://github.com/apache/spark/pull/21052
this time, it were completely different tests (HiveClientSuites) that have
failed.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89596/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89596 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89596/testReport)**
for PR 21052 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89596 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89596/testReport)**
for PR 21052 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21052
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mshtelma commented on the issue:
https://github.com/apache/spark/pull/21052
@gatorsmile the failed tests are not connected to the changes introduced in
this PR. Would it make sense to run the test again ?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21052
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89521/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89521 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89521/testReport)**
for PR 21052 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21052
**[Test build #89521 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89521/testReport)**
for PR 21052 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21052
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mshtelma commented on the issue:
https://github.com/apache/spark/pull/21052
@wzhfy @maropu Hi guys, is there anything else I should add/change to the
PR ?
---
-
To unsubscribe, e-mail:
35 matches
Mail list logo
