Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

YARN-3396. Handle URISyntaxException in ResourceLocalizationService. (Contributed by Brahma Reddy Battula)

This commit is contained in:
Junping Du 2015-05-05 10:18:23 -07:00
parent 05adc76ace
commit 3810242062
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-yarn-project/CHANGES.txt
View File

@ -180,6 +180,8 @@ Release 2.8.0 - UNRELEASED
YARN-3363. add localization and container launch time to ContainerMetrics YARN-3363. add localization and container launch time to ContainerMetrics
at NM to show these timing information for each active container. at NM to show these timing information for each active container.
(zxu via rkanter) (zxu via rkanter)
YARN-3396. Handle URISyntaxException in ResourceLocalizationService.
(Brahma Reddy Battula via junping_du)
OPTIMIZATIONS OPTIMIZATIONS

8
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/localizer/ResourceLocalizationService.java
View File

@ -995,7 +995,9 @@ public class ResourceLocalizationService extends CompositeService
try { try {
req = new LocalResourceRequest(rsrc); req = new LocalResourceRequest(rsrc);
} catch (URISyntaxException e) { } catch (URISyntaxException e) {
// TODO fail? Already translated several times... LOG.error(
"Got exception in parsing URL of LocalResource:"
+ rsrc.getResource(), e);
} }
LocalizerResourceRequestEvent assoc = scheduled.get(req); LocalizerResourceRequestEvent assoc = scheduled.get(req);
if (assoc == null) { if (assoc == null) {
@ -1069,7 +1071,9 @@ public class ResourceLocalizationService extends CompositeService
LOG.error("Inorrect path for PRIVATE localization." LOG.error("Inorrect path for PRIVATE localization."
+ next.getResource().getFile(), e); + next.getResource().getFile(), e);
} catch (URISyntaxException e) { } catch (URISyntaxException e) {
//TODO fail? Already translated several times... LOG.error(
"Got exception in parsing URL of LocalResource:"
+ next.getResource(), e);
} }
} }