Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21068#discussion_r185106176
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -170,8 +170,7 @@ class CoarseGrainedSchedulerBackend(scheduler:
TaskSchedulerImpl, val rpcEnv: Rp
if (executorDataMap.contains(executorId)) {
executorRef.send(RegisterExecutorFailed("Duplicate executor ID:
" + executorId))
context.reply(true)
- } else if (scheduler.nodeBlacklist != null &&
- scheduler.nodeBlacklist.contains(hostname)) {
+ } else if
(scheduler.nodeBlacklistWithExpiryTimes.contains(hostname)) {
--- End diff --
I see Attila fixed this in the tests, but another option would have been to
use `Option(blah).exists()` or something like that.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
