Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/22215
Yup this can merge now, thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user rvesse commented on the issue:
https://github.com/apache/spark/pull/22215
Think this is pretty much ready to merge, can folks take another look when
they get chance
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2789/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
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/22215
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/22215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95616/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
**[Test build #95616 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95616/testReport)**
for PR 22215 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2789/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
**[Test build #95616 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95616/testReport)**
for PR 22215 at commit
Github user rvesse commented on the issue:
https://github.com/apache/spark/pull/22215
@mccheah Thanks for the review, have made the change you suggested to use
N/A instead of empty string.
I have left indentation as tabs for now, as I said in a previous comment
this was just
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
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/22215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95569/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2753/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
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/22215
**[Test build #95569 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95569/testReport)**
for PR 22215 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95568/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
**[Test build #95568 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95568/testReport)**
for PR 22215 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
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/22215
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2753/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
**[Test build #95569 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95569/testReport)**
for PR 22215 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
**[Test build #95568 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95568/testReport)**
for PR 22215 at commit
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/22215
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22215
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/22215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2703/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22215
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2703/
---
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/22215
@mccheah can you give `ok to test` to this one and help merge it?
---
-
To unsubscribe, e-mail:
Github user rvesse commented on the issue:
https://github.com/apache/spark/pull/22215
@liyinan926 @nrchakradhar Addressed all your comments, thanks for the
reviews.
Is someone able to kick off the Jenkins testing on this PR?
---
30 matches
Mail list logo
