Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/1021#discussion_r189997710
--- 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 thought about about the variable context builder idea over lunch. Is
what you are proposing to change this so that `MapVariableResolver`, instead of
taking a list of maps, we take a Map of maps. e.g.
`MapVariableResolver(Map<String, Map<String, Object>> variables)` and the
resolution would check that map first? So, for instance:
```
MapVariableResolver resolver = new MapVariableResolver(ImmutableMap.of("_",
message, "global", globalConfig));
```
---
