This is an automated email from the ASF dual-hosted git repository. jhung pushed a commit to branch YARN-8200 in repository https://gitbox.apache.org/repos/asf/hadoop.git
commit 69b9d76f6f5667cd94db55935def6de26daee5d6 Author: Sunil G <[email protected]> AuthorDate: Mon Dec 4 10:45:07 2017 +0530 YARN-7594. TestNMWebServices#testGetNMResourceInfo fails on trunk. Contributed by Gergely Novák. --- .../yarn/server/nodemanager/webapp/TestNMWebServices.java | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/webapp/TestNMWebServices.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/webapp/TestNMWebServices.java index c9c86b7..8cd8777 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/webapp/TestNMWebServices.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/webapp/TestNMWebServices.java @@ -444,17 +444,17 @@ public class TestNMWebServices extends JerseyTestBase { ClientResponse.class); assertEquals(MediaType.APPLICATION_JSON, response.getType().toString()); - // Access resource-2 should fail (null NMResourceInfo returned). + // Access resource-2 should fail (empty NMResourceInfo returned). JSONObject json = response.getEntity(JSONObject.class); - assertIncludesException(json); + assertEquals(0, json.length()); - // Access resource-3 should fail (unkown plugin) + // Access resource-3 should fail (unknown plugin) response = r.path("ws").path("v1").path("node").path( "resources").path("resource-3").accept(MediaType.APPLICATION_JSON).get( ClientResponse.class); assertEquals(MediaType.APPLICATION_JSON, response.getType().toString()); json = response.getEntity(JSONObject.class); - assertIncludesException(json); + assertEquals(0, json.length()); // Access resource-1 should success response = r.path("ws").path("v1").path("node").path( @@ -522,10 +522,6 @@ public class TestNMWebServices extends JerseyTestBase { assertEquals(2, json.getJSONArray("assignedGpuDevices").length()); } - private void assertIncludesException(JSONObject json) { - assertTrue(json.has("RemoteException")); - } - private void testContainerLogs(WebResource r, ContainerId containerId) throws IOException { final String containerIdStr = containerId.toString(); --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
