Github user arunmahadevan commented on the pull request:

    https://github.com/apache/storm/pull/1217#issuecomment-197523828
  
    @roshannaik setting `topology.eventlogger.executors: 0` completely disables 
event logging by default (i.e. no threads are started to handle event logging). 
If someone tries to turn on event logging in the topology, they wont see any 
events logged and this is a very confusing behavior in my opinion. It makes 
them think that this functionality is totally broken or something is wrong with 
their topology. 
    
    The event logging happens only when they enable it and we don't expect them 
to turn it on all the time. The actual overhead otherwise is [to check if a few 
flags are turned 
on](https://github.com/apache/storm/blob/1.x-branch/storm-core/src/clj/org/apache/storm/daemon/executor.clj#L480).
 
    
    Can you post more details on the topology you ran and the results you 
observed when you set topology.eventlogger.executors to 0 vs setting it to 
`nil` ? 
    
    



---
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