[ https://issues.apache.org/jira/browse/ARTEMIS-3297?focusedWorklogId=597082&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-597082 ]
ASF GitHub Bot logged work on ARTEMIS-3297: ------------------------------------------- Author: ASF GitHub Bot Created on: 15/May/21 04:13 Start Date: 15/May/21 04:13 Worklog Time Spent: 10m Work Description: clebertsuconic commented on a change in pull request #3580: URL: https://github.com/apache/activemq-artemis/pull/3580#discussion_r632892149 ########## File path: artemis-journal/src/main/java/org/apache/activemq/artemis/core/journal/impl/JournalImpl.java ########## @@ -1858,6 +1988,8 @@ public synchronized void compact() throws Exception { journalLock.writeLock().unlock(); } + processBackup(); Review comment: processBackup() will copy the BKP files to the retention folder. Before I do the computation I need any BKP files cleared so they can be treated. So, the backup is before compacting, hence I'm calling processBackup before compacting here. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 597082) Time Spent: 7h 10m (was: 7h) > Journal Retention Feature > ------------------------- > > Key: ARTEMIS-3297 > URL: https://issues.apache.org/jira/browse/ARTEMIS-3297 > Project: ActiveMQ Artemis > Issue Type: New Feature > Affects Versions: 2.17.0 > Reporter: Clebert Suconic > Priority: Major > Fix For: 2.18.0 > > Time Spent: 7h 10m > Remaining Estimate: 0h > > We should keep historical backup of the journal data, and having tools to > recover data in case of data loss. > > Data loss could mean by unbehaved client's (say you mistakenly acknowledged > data and your code caused the message to disappear earlier), or even after > eventual bugs on the broker. > > To activate the functionality, you have to specify journal-history-data on > broker.xml. -- This message was sent by Atlassian Jira (v8.3.4#803005)