Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/23223
Hi @tgravescs , I tried it, but found it's difficult to produce
KILLED_BY_RESOURCEMANAGER exit status. I followed
[YARN-73](https://issues.apache.org/jira/browse/YARN-73)
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/23223
> it would be interesting to test it further to see if it does.
@tgravescs Yeah, I have the same thought. I'd like to try it, but I can
not guarantee that I can achieve it... Because I
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23223
if you aren't seeing actual issues with this I guess it would be
interesting to test it further to see if it does. I can see spark blacklisting
when it shouldn't for exit codes like you mention
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23223
**[Test build #99757 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99757/testReport)**
for PR 23223 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99757/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
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/23223
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
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/23223
**[Test build #99757 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99757/testReport)**
for PR 23223 at commit
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/23223
> it looks like its only going to blacklist the node for the AM, not other
nodes for general containers.
@squito Yarn have blacklist for AM when config
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/23223
> Are you seeing actual issues with this blacklisting when it shouldn't?
Unfortunately, no. @tgravescs @squito
---
-
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/23223
> I mean if node blacklisting in Spark would be perfectly aligned to YARN
then it would be just redundant to have it in Spark in the first place.
This change seems result in *perfectly*
Github user squito commented on the issue:
https://github.com/apache/spark/pull/23223
@attilapiros
> I mean if node blacklisting in Spark would be perfectly aligned to YARN
then it would be just redundant to have it in Spark in the first place.
I'm not super
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99733/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23223
**[Test build #99733 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99733/testReport)**
for PR 23223 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23223
the approach here makes sense. Are you seeing actual issues with this
blacklisting when it shouldn't? I could see that possible there and if so we
should move this to defect and make sure it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
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/23223
**[Test build #99733 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99733/testReport)**
for PR 23223 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23223
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23223
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/23223
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/23223
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
23 matches
Mail list logo
