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

    https://github.com/apache/metron/pull/1021#discussion_r189905091
  
    --- 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 --
    
    I don't doubt the need or applicability, I guess I'm saying why not just 
have the _ in the configuration side, and just have the scripts reference the 
vars by name and not have to MAP_GET()?
    
    That way, the vars are the vars are the vars in Stellar, and the external 
configuration can have the option for putting all the message in the var 
namespace.
    



---

Reply via email to