Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/851#discussion_r154207614
--- 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 --
We need to refactor `ConfigurationUtils` so badly that it hurts.
---
