Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/22857
Thanks all for reviewing! The latest change looks good to me too. Merged
into master.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
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/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98317/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22857
**[Test build #98317 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98317/testReport)**
for PR 22857 at commit
[`5499651`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
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/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4665/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22857
LGTM except the end-to-end test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22857
Please be really careful in null handling. It could easily introduce the
correctness bugs like what we recently fixed.
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98239/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22857
**[Test build #98239 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98239/testReport)**
for PR 22857 at commit
[`0eac890`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98240/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
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/22857
**[Test build #98240 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98240/testReport)**
for PR 22857 at commit
[`4c35955`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22857
**[Test build #98240 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98240/testReport)**
for PR 22857 at commit
[`4c35955`](https://github.com/apache/spark/commit/4c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4615/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
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/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4614/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22857
**[Test build #98239 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98239/testReport)**
for PR 22857 at commit
[`0eac890`](https://github.com/apache/spark/commit/0e
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/22857
LGTM.
@cloud-fan and @gatorsmile, this is the PR I mentioned to you earlier this
year in the SF Spark summit which can simplify some of our queries.
Also add @dongjoon-hyun and @vi
Github user aokolnychyi commented on the issue:
https://github.com/apache/spark/pull/22857
@dbtsai @gatorsmile @cloud-fan could you guys, please, take a look?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98127/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
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/22857
**[Test build #98127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98127/testReport)**
for PR 22857 at commit
[`1d8fefd`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22857
**[Test build #98127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98127/testReport)**
for PR 22857 at commit
[`1d8fefd`](https://github.com/apache/spark/commit/1d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22857
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/22857
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4555/
28 matches
Mail list logo