Github user merrimanr commented on the issue:

    https://github.com/apache/incubator-metron/pull/338
  
    @rkarthik29 One of the advantages of keeping the configuration (scripts in 
this case) in Zookeeper is that you get notified when something changes.  You 
don't need to track versions. Unfortunately, until #308 gets merged in, the 
ParserBolt does not notify the parser class that the config has changed and 
only provides the configuration on startup (hence the need to restart the 
topology).
    
    I think it makes sense to keep it in HDFS for this PR and accept the 
current restriction of having to restart the topology for it to take effect.  
Until we merge #308 or agree on the another approach, I wouldn't worry about 
script changes being updated on the fly.  There has already been a lot of 
discussion and work put into this so it doesn't make sense to start over.  We 
could even update #308 to include this parser once we get closer to merging 
that.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to