Github user squito commented on the issue:
https://github.com/apache/spark/pull/21475
merged to master. Thanks @redsanket !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21475
+1
---
-
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/21475
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91712/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21475
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/21475
**[Test build #91712 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91712/testReport)**
for PR 21475 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21475
**[Test build #91712 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91712/testReport)**
for PR 21475 at commit
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21475
lgtm
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21475
**[Test build #91667 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91667/testReport)**
for PR 21475 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21475
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/21475
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91667/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21475
**[Test build #91667 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91667/testReport)**
for PR 21475 at commit
Github user redsanket commented on the issue:
https://github.com/apache/spark/pull/21475
@tgravescs @squito fixed the description hope you can take a look thanks
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21475
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21475
that wording for the application level sounds good, so we are talking a
about changing it to be something like:
If set to "true", allow Spark to automatically kill the
executors
Github user redsanket commented on the issue:
https://github.com/apache/spark/pull/21475
That description might work too @squito If that is ok with @tgravescs I can
rephrase it... Just wanted it to be explicit
---
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21475
what about "... when they are blacklisted for the entire application, as
controlled by spark.blacklist.application.*, ..."?
---
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21475
yes I believe so. I think the main thing is to just clarify it so that
users don't expect it to be killed if its just blacklisted for the stage. It
has to be propagated up to the application
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21475
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91381/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21475
**[Test build #91381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91381/testReport)**
for PR 21475 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21475
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21475
I think original description is OK, just not too detailed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21475
**[Test build #91381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91381/testReport)**
for PR 21475 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21475
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/21475
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21475
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user redsanket commented on the issue:
https://github.com/apache/spark/pull/21475
@tgravescs plz review thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
26 matches
Mail list logo