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

    https://github.com/apache/metron/pull/1213#discussion_r220987950
  
    --- Diff: 
metron-platform/metron-parsers/src/main/java/org/apache/metron/parsers/bolt/ParserBolt.java
 ---
    @@ -242,169 +226,81 @@ public void prepare(Map stormConf, TopologyContext 
context, OutputCollector coll
         }
       }
     
    -  protected void initializeStellar() {
    -    Context.Builder builder = new Context.Builder()
    -                                
.with(Context.Capabilities.ZOOKEEPER_CLIENT, () -> client)
    -                                .with(Context.Capabilities.GLOBAL_CONFIG, 
() -> getConfigurations().getGlobalConfig())
    -                                .with(Context.Capabilities.STELLAR_CONFIG, 
() -> getConfigurations().getGlobalConfig())
    -                                ;
    -    if(cache != null) {
    -      builder = builder.with(Context.Capabilities.CACHE, () -> cache);
    -    }
    -    this.stellarContext = builder.build();
    -    StellarFunctions.initialize(stellarContext);
    -  }
    -
     
       @SuppressWarnings("unchecked")
       @Override
       public void execute(Tuple tuple) {
         if (TupleUtils.isTick(tuple)) {
    -      try {
    --- End diff --
    
    Slight refactor here.  I moved this to it's own method to make execute more 
readable.


---

Reply via email to