Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/851#discussion_r154207266
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/ConfigurationsUtils.java
---
@@ -156,7 +156,7 @@ public static void
writeConfigToZookeeper(ConfigurationType configType,Optional<
}
private static String getConfigZKPath(ConfigurationType configType,
Optional<String> configName) {
- String pathSuffix = configName.isPresent() && configType != GLOBAL ?
"/" + configName : "";
+ String pathSuffix = configName.isPresent() && configType != GLOBAL ?
"/" + configName.get() : "";
--- End diff --
Yeah, this confused me too. There are a lot of paths through
ConfigurationUtils. Some paths worked, others didn't.
---
