This is an automated email from the ASF dual-hosted git repository. ayushsaxena pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/trunk by this push: new e0938b4c2aa YARN-11495. Fix typos in hadoop-yarn-server-web-proxy. (#5652). Contributed by Shilun Fan. e0938b4c2aa is described below commit e0938b4c2aa5120280023fb5985ec7508d2a2ba3 Author: slfan1989 <55643692+slfan1...@users.noreply.github.com> AuthorDate: Sat May 13 14:11:38 2023 +0800 YARN-11495. Fix typos in hadoop-yarn-server-web-proxy. (#5652). Contributed by Shilun Fan. Signed-off-by: Ayush Saxena <ayushsax...@apache.org> --- .../org/apache/hadoop/yarn/server/webproxy/AppReportFetcher.java | 2 +- .../java/org/apache/hadoop/yarn/server/webproxy/ProxyUriUtils.java | 4 ++-- .../org/apache/hadoop/yarn/server/webproxy/WebAppProxyServer.java | 2 +- .../org/apache/hadoop/yarn/server/webproxy/WebAppProxyServlet.java | 2 +- .../apache/hadoop/yarn/server/webproxy/TestWebAppProxyServlet.java | 6 +++--- 5 files changed, 8 insertions(+), 8 deletions(-) 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/AppReportFetcher.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/AppReportFetcher.java index 6b79e194b41..94c34da031c 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/AppReportFetcher.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/AppReportFetcher.java @@ -148,7 +148,7 @@ public class AppReportFetcher { /* * This class creates a bundle of the application report and the source from - * where the the report was fetched. This allows the WebAppProxyServlet + * where the report was fetched. This allows the WebAppProxyServlet * to make decisions for the application report based on the source. */ static class FetchedAppReport { 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/ProxyUriUtils.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/ProxyUriUtils.java index f25f4c7bb72..fbd5449a94a 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/ProxyUriUtils.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/ProxyUriUtils.java @@ -164,13 +164,13 @@ public class ProxyUriUtils { originalUri == null ? null : originalUri.getQuery(), originalUri == null ? null : originalUri.getFragment()); } catch (URISyntaxException e) { - throw new RuntimeException("Could not proxify "+originalUri,e); + throw new RuntimeException("Could not proxy "+originalUri, e); } } /** * Create a URI form a no scheme Url, such as is returned by the AM. - * @param noSchemeUrl the URL formate returned by an AM + * @param noSchemeUrl the URL format returned by an AM * @return a URI with an http scheme * @throws URISyntaxException if the url is not formatted correctly. */ 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/WebAppProxyServer.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/WebAppProxyServer.java index 6be6039e6d0..a62ba18a6e5 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/WebAppProxyServer.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/WebAppProxyServer.java @@ -86,7 +86,7 @@ public class WebAppProxyServer extends CompositeService { } /** - * Log in as the Kerberose principal designated for the proxy + * Log in as the Kerberos principal designated for the proxy * @param conf the configuration holding this information in it. * @throws IOException on any error. */ 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/WebAppProxyServlet.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/WebAppProxyServlet.java index c0187ea08d3..75b891b7206 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/WebAppProxyServlet.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/main/java/org/apache/hadoop/yarn/server/webproxy/WebAppProxyServlet.java @@ -103,7 +103,7 @@ public class WebAppProxyServlet extends HttpServlet { /** * HTTP methods. */ - private enum HTTP { GET, POST, HEAD, PUT, DELETE }; + private enum HTTP { GET, POST, HEAD, PUT, DELETE } /** * Empty Hamlet class. diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/test/java/org/apache/hadoop/yarn/server/webproxy/TestWebAppProxyServlet.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/test/java/org/apache/hadoop/yarn/server/webproxy/TestWebAppProxyServlet.java index c59d5b952a8..39699ae7d90 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/test/java/org/apache/hadoop/yarn/server/webproxy/TestWebAppProxyServlet.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy/src/test/java/org/apache/hadoop/yarn/server/webproxy/TestWebAppProxyServlet.java @@ -345,7 +345,7 @@ public class TestWebAppProxyServlet { AppReportFetcherForTest appReportFetcher = proxy.proxy.appReportFetcher; try { - //set AHS_ENBALED = false to simulate getting the app report from RM + //set AHS_ENABLED = false to simulate getting the app report from RM configuration.setBoolean(YarnConfiguration.APPLICATION_HISTORY_ENABLED, false); ApplicationId app = ApplicationId.newInstance(0, 0); @@ -366,7 +366,7 @@ public class TestWebAppProxyServlet { "/cluster" + "/app/" + app.toString(); assertEquals(proxyConn.getURL().toString(), appAddressInRm); - //set AHS_ENBALED = true to simulate getting the app report from AHS + //set AHS_ENABLED = true to simulate getting the app report from AHS configuration.setBoolean(YarnConfiguration.APPLICATION_HISTORY_ENABLED, true); proxyConn = (HttpURLConnection) url.openConnection(); @@ -655,7 +655,7 @@ public class TestWebAppProxyServlet { /* * If this method is called with isTrackingUrl=false, no tracking url * will set in the app report. Hence, there will be a connection exception - * when the prxyCon tries to connect. + * when the proxyCon tries to connect. */ private FetchedAppReport getDefaultApplicationReport(ApplicationId appId, boolean isTrackingUrl) { --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org