Github user arunmahadevan commented on the pull request:

    https://github.com/apache/storm/pull/644#issuecomment-125086304
  
    @harshach Are you saying that we don't support time based rotation at all? 
The current approach automatically turns off exactly once if TimedRotation is 
in use, so that who want to use time based rotation would still be able to use 
it with the limitation that they will not get exactly once semantics.
    
    I also think we can do the same with max file size in case of size based 
rotation policy.
    



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