Ironankit525 opened a new pull request, #8415:
URL: https://github.com/apache/storm/pull/8415

   ## Its a feature request
   
   ## What is the purpose of the change
   
   The `FileBasedEventLogger` currently has no bounds on the amount of data it 
writes to `events.log`, which can lead to disk exhaustion over time for 
long-running topologies. There is a `//TODO: file rotation` comment in the code 
indicating this was planned.
   
   This PR implements file rotation for the `FileBasedEventLogger` based on 
file size and retained file count.
   
   ## How was this change tested
   
   Added `FileBasedEventLoggerTest.java` to test that files correctly rotate 
after reaching the size limit and older files are deleted when exceeding the 
max retained files. Tests execute properly and rotation chunks are verified.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to