Github user cestella commented on a diff in the pull request:

    https://github.com/apache/metron/pull/1021#discussion_r189932382
  
    --- Diff: 
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/enrichment/handler/StellarConfig.java
 ---
    @@ -142,8 +143,14 @@ else if(kv.getValue() instanceof List) {
       {
     
    --- End diff --
    
    The variables referenced are spread across multiple maps.  It's 
unreasonable to union those maps together for every message, rather passing 
back the ConfigMap, which is a lazy facade on Map that doesn't union the maps.  
The variable resolver can choose to support `_` or not, so the support has to 
happen in the variable resolver.


---

Reply via email to