YARN-4530. LocalizedResource trigger a NPE Cause the NodeManager exit. (tangshangwen via rohithsharmaks)
This commit is contained in:
parent
4e4b3a8465
commit
f9e36dea96
|
@ -99,6 +99,9 @@ Release 2.9.0 - UNRELEASED
|
||||||
YARN-2934. Improve handling of container's stderr.
|
YARN-2934. Improve handling of container's stderr.
|
||||||
(Naganarasimha G R via gera)
|
(Naganarasimha G R via gera)
|
||||||
|
|
||||||
|
YARN-4530. LocalizedResource trigger a NPE Cause the NodeManager exit
|
||||||
|
(tangshangwen via rohithsharmaks)
|
||||||
|
|
||||||
Release 2.8.0 - UNRELEASED
|
Release 2.8.0 - UNRELEASED
|
||||||
|
|
||||||
INCOMPATIBLE CHANGES
|
INCOMPATIBLE CHANGES
|
||||||
|
|
|
@ -878,12 +878,12 @@ public class ResourceLocalizationService extends CompositeService
|
||||||
Future<Path> completed = queue.take();
|
Future<Path> completed = queue.take();
|
||||||
LocalizerResourceRequestEvent assoc = pending.remove(completed);
|
LocalizerResourceRequestEvent assoc = pending.remove(completed);
|
||||||
try {
|
try {
|
||||||
Path local = completed.get();
|
|
||||||
if (null == assoc) {
|
if (null == assoc) {
|
||||||
LOG.error("Localized unknown resource to " + completed);
|
LOG.error("Localized unknown resource to " + completed);
|
||||||
// TODO delete
|
// TODO delete
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
Path local = completed.get();
|
||||||
LocalResourceRequest key = assoc.getResource().getRequest();
|
LocalResourceRequest key = assoc.getResource().getRequest();
|
||||||
publicRsrc.handle(new ResourceLocalizedEvent(key, local, FileUtil
|
publicRsrc.handle(new ResourceLocalizedEvent(key, local, FileUtil
|
||||||
.getDU(new File(local.toUri()))));
|
.getDU(new File(local.toUri()))));
|
||||||
|
|
Loading…
Reference in New Issue