Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19515
LGTM, merging to master / 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19515
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83648/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19515
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/19515
**[Test build #83648 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83648/testReport)**
for PR 19515 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19515
@pmackles perhaps you could email this to d...@spark.apache.org to get some
visibility to this and hopefully someone else on the mesos side can review?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19515
**[Test build #83648 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83648/testReport)**
for PR 19515 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19515
@susanxhuynh or anyone from the mesos side would you please review?
---
-
To unsubscribe, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19515
Jenkins, test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19515
@felixcheung - any chance of getting this tiny change merged and included
in the upcoming 2.2.1 release?
---
-
To unsubscribe,
Github user ArtRand commented on the issue:
https://github.com/apache/spark/pull/19515
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19515
@ArtRand - yeah default would remain the same which is 1g
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user ArtRand commented on the issue:
https://github.com/apache/spark/pull/19515
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ArtRand commented on the issue:
https://github.com/apache/spark/pull/19515
Yeah that sounds good. Thanks for this!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19515
@ArtRand - WDYT? I was going to switch it to ```SPARK_DISPATCHER_MEMORY```
but then I noticed that the other env vars for MesosClusterDispatcher or also
prefixed with ```SPARK_DAEMON_*``` so I
Github user pmackles commented on the issue:
https://github.com/apache/spark/pull/19515
Hi @ArtRand - Based on my testing and interpretation of the code,
```SPARK_DRIVER_MEMORY``` has no affect on MesosClusterDispatcher. Heap size
always winds up being set to ```-Xmx1G``` (the
Github user ArtRand commented on the issue:
https://github.com/apache/spark/pull/19515
Hello @pmackles, thanks for this. It would be set to the value of
`SPARK_DRIVER_MEMORY` by default correct? What to you think about introducing a
new envvar (`SPARK_DISPATCHER_MEMORY`) so that it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19515
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
17 matches
Mail list logo