franz1981 edited a comment on pull request #3573:
URL: https://github.com/apache/activemq-artemis/pull/3573#issuecomment-914053249


   TLDR on this change (still need to run CI on this): 
   
   - lock-free sync requests (no interference with background sync):  it 
affects `PagingStoreImpl::page`, that's performing `sync` on
   ```java
               if (tx == null && syncNonTransactional && message.isDurable()) {
                  sync();
               }
   ```
   - performing background sync no longer allocate ` OperationContext[]`: it 
affects paging with fast disks ie with a very short page sync timeout
   
   No hurry to get it in, but I think is a nice improvement on paging 
@clebertsuconic @michaelandrepearce wdyt?


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