Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20645
> A quick question: after this change, extraJavaOptions is still able to
cleanly override whatever's set in defaultJavaOptions, is that right?
No, the intent is for both sets of options to
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/20645
A quick question: after this change, `extraJavaOptions` is still able to
cleanly override whatever's set in `defaultJavaOptions`, is that right?
^^ Just making sure I understood the
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20645
I like the `ConfigBuilder` approach. That would make this much more useful.
I'll add an implementation like that.
I think append option syntax would be confusing for users and
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20645
I agree it would be nicer to have this be a more general feature. I would
prefer an approach which didn't require a different configuration name, just as
its more to document & for users to keep
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20645
I like the functionality, not necessarily the implementation. Other configs
would benefit from this, and having this functionality more easily available
without having to change the call sites would
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20645
@vanzin, you might be interested in this one because it makes applying
administrator settings for Spark much easier.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20645
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87569/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20645
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/20645
**[Test build #87569 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87569/testReport)**
for PR 20645 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20645
**[Test build #87569 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87569/testReport)**
for PR 20645 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20645
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/975/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20645
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
12 matches
Mail list logo