Skip to content

fix: resolve log file cleanup issue on BifroMQ restart#197

Merged
popduke merged 2 commits intoapache:mainfrom
liaodn:main
Jan 29, 2026
Merged

fix: resolve log file cleanup issue on BifroMQ restart#197
popduke merged 2 commits intoapache:mainfrom
liaodn:main

Conversation

@liaodn
Copy link
Contributor

@liaodn liaodn commented Dec 4, 2025

  • Add OnStartupTriggeringPolicy to trigger cleanup on startup
  • Implement explicit Delete strategy with 72h or 7h retention
  • Prevent log file accumulation and disk space exhaustion

Fixes #196

- Add OnStartupTriggeringPolicy to trigger cleanup on startup
- Implement explicit Delete strategy with 72h or 7h retention
- Prevent log file accumulation and disk space exhaustion
@coveralls
Copy link

coveralls commented Dec 5, 2025

Pull Request Test Coverage Report for Build 21442573341

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 57 unchanged lines in 7 files lost coverage.
  • Overall coverage decreased (-0.06%) to 65.156%

Files with Coverage Reduction New Missed Lines %
org/apache/bifromq/basecluster/memberlist/agent/Agent.java 1 95.45%
org/apache/bifromq/basecluster/transport/TCPTransport.java 1 87.06%
org/apache/bifromq/basekv/raft/exception/LeaderTransferException.java 3 81.25%
org/apache/bifromq/dist/trie/STopicFilterTrieNode.java 3 88.28%
org/apache/bifromq/basekv/raft/RaftNodeStateLeader.java 4 86.05%
org/apache/bifromq/dist/trie/TopicFilterIterator.java 6 72.89%
org/apache/bifromq/basekv/store/range/KVRangeFSM.java 39 80.37%
Totals Coverage Status
Change from base Build 21442561745: -0.06%
Covered Lines: 27863
Relevant Lines: 41280

💛 - Coveralls

@popduke
Copy link
Contributor

popduke commented Dec 11, 2025

looks good to me, will merge after first release out

@popduke popduke merged commit 57bba39 into apache:main Jan 29, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Log4j2 RollingFileAppender with TimeBasedTriggeringPolicy fails to delete old logs on application restart

3 participants