[ 
https://issues.apache.org/jira/browse/METRON-1520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434517#comment-16434517
 ] 

ASF GitHub Bot commented on METRON-1520:
----------------------------------------

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

    https://github.com/apache/metron/pull/990#discussion_r180885231
  
    --- Diff: 
metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/json/JSONMapParser.java
 ---
    @@ -99,6 +98,7 @@ public void configure(Map<String, Object> config) {
         String strategyStr = (String) config.getOrDefault(MAP_STRATEGY_CONFIG, 
MapStrategy.DROP.name());
         mapStrategy = MapStrategy.valueOf(strategyStr);
         if (config.containsKey(JSONP_QUERY)) {
    --- End diff --
    
    Sorry, I wasn't finished with the PRs submitted.  This is not strictly 
necessary except that I ran into the bug as part of testing this PR.  I 
submitted a separate PR for this fix in #991 and will remove this.


> Add caching for stellar field transformations
> ---------------------------------------------
>
>                 Key: METRON-1520
>                 URL: https://issues.apache.org/jira/browse/METRON-1520
>             Project: Metron
>          Issue Type: Improvement
>            Reporter: Casey Stella
>            Priority: Major
>
> Given how important caching is in the enrichment topology, we should have 
> caching for stellar field transformations in the parsers as well.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to