Github user nickwallen commented on the issue:

    https://github.com/apache/incubator-metron/pull/449
  
    > Ninja Edit: I think the kafka topic written to should be pulled from 
zookeeper...
    
    @cestella I remember now why I settled on making the topic name a static 
configuration from the topology properties.  Our `BulkMessageWriterBolt` & 
`KafkaMessageWriter` classes seem to have been designed to only work with 
statically defined topic names.  I would have to change those core classes, if 
I want to set the topic name dynamically from Zookeeper.  And I tend to get 
smacked around when coming close to core classes :)
    
    Am I missing something?  Is there a way to define the topic dynamically 
while using the `BulkMessageWriterBolt` & `KafkaMessageWriter` classes 
unchanged?
    
    If not, I'd prefer to keep the topic name as a static property defined in 
the topology properties, at least for this PR. My second choice would be to 
open a completely separate PR to update those core classes to accept dynamic 
topic names.  It does not seem like a complex change, but those are core 
classes and will likely cause some debate.
    
    
    



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to