YARN-7801. AmFilterInitializer should addFilter after fill all parameters. Contributed by Wangda Tan
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/55c32776 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/55c32776 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/55c32776 Branch: refs/heads/YARN-1011 Commit: 55c32776b1f130c6b81ef7aff5fc1562180f4601 Parents: 9760761 Author: Jian He <[email protected]> Authored: Wed Jan 24 11:16:54 2018 -0800 Committer: Jian He <[email protected]> Committed: Wed Jan 24 11:17:12 2018 -0800 ---------------------------------------------------------------------- .../hadoop/yarn/server/webproxy/amfilter/AmFilterInitializer.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/55c32776/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/amfilter/AmFilterInitializer.java ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/amfilter/AmFilterInitializer.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/amfilter/AmFilterInitializer.java index c3ddc54..27742e4 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/amfilter/AmFilterInitializer.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/amfilter/AmFilterInitializer.java @@ -59,7 +59,6 @@ public class AmFilterInitializer extends FilterInitializer { } sb.setLength(sb.length() - 1); params.put(AmIpFilter.PROXY_URI_BASES, sb.toString()); - container.addFilter(FILTER_NAME, FILTER_CLASS, params); // Handle RM HA urls // Include yarn-site.xml in the classpath @@ -75,6 +74,8 @@ public class AmFilterInitializer extends FilterInitializer { params.put(RM_HA_URLS, StringUtils.join(",", urls)); } } + + container.addFilter(FILTER_NAME, FILTER_CLASS, params); } private Collection<String> getRmIds(Configuration conf) { --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
