[GitHub] spark pull request #21104: [SPARK-24021][CORE] fix bug in BlacklistTracker's...

2018-04-19 Thread asfgit 
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/21104


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #21104: [SPARK-24021][CORE] fix bug in BlacklistTracker's...

2018-04-18 Thread Ngone51 
GitHub user Ngone51 opened a pull request:

https://github.com/apache/spark/pull/21104

[SPARK-24021][CORE] fix bug in BlacklistTracker's 
updateBlacklistForFetchFailure

## What changes were proposed in this pull request?

There‘s a miswrite in BlacklistTracker's updateBlacklistForFetchFailure:
```
val blacklistedExecsOnNode =
nodeToBlacklistedExecs.getOrElseUpdate(exec, HashSet[String]())
blacklistedExecsOnNode += exec
```
where first **exec** should be **host**.
## How was this patch tested?

adjust existed test.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Ngone51/spark SPARK-24021

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/21104.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #21104


commit d2257213ecd4b0e8ec91bfa52f7caf725c267b16
Author: wuyi 
Date:   2018-04-19T02:42:51Z

fix bug in BlacklistTracker's updateBlacklistForFetchFailure




---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org