Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20812
Should we close this then? @jinxing64 @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/20812
@jerryshao
Understood, `Ideally different udfs should be packaged in different jars
with different name/version`. True. But we are faced with tons of udf/jars
migrating from other engine. I m
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/20812
@jinxing64 , I think using same name jars which contains different classes
seems practically is not a best practice. Ideally different udfs should be
packaged in different jars with different name
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88302/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
**[Test build #88302 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88302/testReport)**
for PR 20812 at commit
[`4ac1e8e`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88303/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20812
**[Test build #88303 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88303/testReport)**
for PR 20812 at commit
[`4473878`](https://github.com/apache/spark/commit/4
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/20812
@jerryshao
Thanks for comment;
Yes, this change is only for `sc.addJar` and the jars will be named with a
prefix when executor `updateDependencies`.
---
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/20812
Does it only fix the jars added by `sc.addJar` or using non-yarn mode?
Because yarn uses distributed cache at start, so it has a different code path,
right?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20812
**[Test build #88303 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88303/testReport)**
for PR 20812 at commit
[`4473878`](https://github.com/apache/spark/commit/44
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/1558/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
**[Test build #88302 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88302/testReport)**
for PR 20812 at commit
[`4ac1e8e`](https://github.com/apache/spark/commit/4a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
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/1557/
Tes
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/20812
@jiangxb1987
Thanks a lot for review. I will refine soon !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20812
The idea looks good, just a few comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/20812
@vanzin @zsxwing @jerryshao
How do you think about this ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88208/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20812
**[Test build #88208 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88208/testReport)**
for PR 20812 at commit
[`f78c273`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/1489/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
**[Test build #88208 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88208/testReport)**
for PR 20812 at commit
[`f78c273`](https://github.com/apache/spark/commit/f7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20812
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/20812
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/1488/
Tes
28 matches
Mail list logo