KNOX-1124 - Service Discovery Doesn't Work For Simple Descriptors With .yaml Extension (Phil Zampino via Sandeep More)
Project: http://git-wip-us.apache.org/repos/asf/knox/repo Commit: http://git-wip-us.apache.org/repos/asf/knox/commit/a09e751d Tree: http://git-wip-us.apache.org/repos/asf/knox/tree/a09e751d Diff: http://git-wip-us.apache.org/repos/asf/knox/diff/a09e751d Branch: refs/heads/KNOX-998-Package_Restructuring Commit: a09e751d8e9d3a5f8a8989dec11f96d517bc16b5 Parents: 7b21146 Author: Sandeep More <[email protected]> Authored: Fri Dec 1 13:43:10 2017 -0500 Committer: Sandeep More <[email protected]> Committed: Fri Dec 1 13:43:10 2017 -0500 ---------------------------------------------------------------------- .../topology/simple/SimpleDescriptorFactory.java | 2 +- .../topology/simple/SimpleDescriptorFactoryTest.java | 13 +++++++++++-- 2 files changed, 12 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/knox/blob/a09e751d/gateway-server/src/main/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactory.java ---------------------------------------------------------------------- diff --git a/gateway-server/src/main/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactory.java b/gateway-server/src/main/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactory.java index 3df6d2f..2192eef 100644 --- a/gateway-server/src/main/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactory.java +++ b/gateway-server/src/main/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactory.java @@ -39,7 +39,7 @@ public class SimpleDescriptorFactory { if (path.endsWith(".json")) { sd = parseJSON(path); - } else if (path.endsWith(".yml")) { + } else if (path.endsWith(".yml") || path.endsWith(".yaml")) { sd = parseYAML(path); } else { throw new IllegalArgumentException("Unsupported simple descriptor format: " + path.substring(path.lastIndexOf('.'))); http://git-wip-us.apache.org/repos/asf/knox/blob/a09e751d/gateway-server/src/test/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactoryTest.java ---------------------------------------------------------------------- diff --git a/gateway-server/src/test/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactoryTest.java b/gateway-server/src/test/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactoryTest.java index 129ff7c..7c42542 100644 --- a/gateway-server/src/test/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactoryTest.java +++ b/gateway-server/src/test/java/org/apache/hadoop/gateway/topology/simple/SimpleDescriptorFactoryTest.java @@ -29,7 +29,8 @@ public class SimpleDescriptorFactoryTest { private enum FileType { JSON, - YAML + YAML, + YML } @Test @@ -39,6 +40,7 @@ public class SimpleDescriptorFactoryTest { @Test public void testParseYAMLSimpleDescriptor() throws Exception { + testParseSimpleDescriptor(FileType.YML); testParseSimpleDescriptor(FileType.YAML); } @@ -49,6 +51,7 @@ public class SimpleDescriptorFactoryTest { @Test public void testParseYAMLSimpleDescriptorWithServiceParams() throws Exception { + testParseSimpleDescriptorWithServiceParams(FileType.YML); testParseSimpleDescriptorWithServiceParams(FileType.YAML); } @@ -59,6 +62,7 @@ public class SimpleDescriptorFactoryTest { @Test public void testParseYAMLSimpleDescriptorApplications() throws Exception { + testParseSimpleDescriptorWithApplications(FileType.YML); testParseSimpleDescriptorWithApplications(FileType.YAML); } @@ -70,6 +74,7 @@ public class SimpleDescriptorFactoryTest { @Test public void testParseYAMLSimpleDescriptorWithServicesAndApplications() throws Exception { + testParseSimpleDescriptorWithServicesAndApplications(FileType.YML); testParseSimpleDescriptorWithServicesAndApplications(FileType.YAML); } @@ -327,9 +332,12 @@ public class SimpleDescriptorFactoryTest { case JSON: extension = "json"; break; - case YAML: + case YML: extension = "yml"; break; + case YAML: + extension = "yaml"; + break; } return extension; } @@ -402,6 +410,7 @@ public class SimpleDescriptorFactoryTest { appParams); break; case YAML: + case YML: result = writeYAML(path, discoveryType, discoveryAddress,
