Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21555
Will follow-up. Thanks all for the comments! Merging to master.
---
-
To unsubscribe, e-mail:
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21555
Yup feel free to merge and follow up separately.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/21555
LGMT, I am OK to merge.
Most of the automated image build tooling I've seen is custom, but I agree
w/ Matt that being able to selectively build is worth supporting, via a
followup
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21555
@mccheah, that is a good point but I agree with @ifilonenko that we can do
it in a subsequent PR. I'm thinking we merge this as-is and I can try to get a
follow-up PR here for dockerfile refactor.
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21555
@mccheah Good note. As this is a blocker for other PRs. It is probably best
to refactor the `docker-image-tool.sh` in a separate PR for that is the not the
focus of this PR.
---
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21555
This change makes it such that using the tool forces building and pushing
both Python and non-Python, but, what if the user wants to only build one to
save time? I can imagine that being the case
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21555
Lgtm. Will merge when tests pass. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21555
Good to merge with tests success. @mccheah and @foxish please merge
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92103/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/21555
**[Test build #92103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92103/testReport)**
for PR 21555 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/330/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/330/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/21555
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/21555
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/4224/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/330/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
**[Test build #92103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92103/testReport)**
for PR 21555 at commit
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21555
retest this please
---
-
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/21555
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92063/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/21555
**[Test build #92063 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92063/testReport)**
for PR 21555 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/21555
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92054/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
**[Test build #92054 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92054/testReport)**
for PR 21555 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/307/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
Test FAILed.
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/307/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/21555
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/21555
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/4201/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/307/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
**[Test build #92063 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92063/testReport)**
for PR 21555 at commit
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21555
retest this please
---
-
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/21555
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/4193/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
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/21555
Test FAILed.
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/299/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21555
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/299/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/299/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21555
**[Test build #92054 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92054/testReport)**
for PR 21555 at commit
Github user ifilonenko commented on the issue:
https://github.com/apache/spark/pull/21555
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
41 matches
Mail list logo