[ https://issues.apache.org/jira/browse/OAK-5626?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867847#comment-15867847 ]
Vikas Saurabh commented on OAK-5626: ------------------------------------ bq. "Suppressing further such cases for 600 sec" Good point, would add it. Btw, I don't know the backport-ability - should this go back? I'm biased a bit here as I had to spend at least 3 hours traversing ChangeProcessor while investigating a setup where everything looked like queue was full but the warning was absent from the logs (I didn't immediately had access to Consolidated Event Listener Stats). > ChangeProcessor doesn't reset 'blocking' flag when items from queue gets > removed and commit-rate-limiter is null > ---------------------------------------------------------------------------------------------------------------- > > Key: OAK-5626 > URL: https://issues.apache.org/jira/browse/OAK-5626 > Project: Jackrabbit Oak > Issue Type: Bug > Components: core > Reporter: Vikas Saurabh > Assignee: Vikas Saurabh > Priority: Minor > Attachments: OAK-5626.patch, OAK-5626-v2.patch, OAK-5626-v3.patch > > > Following up on conversation at \[0]: > {{ChangeProcessor#queueSizeChanged}} \[1] sets blocking flag to true if queue > size is hit (or beyond). The warning "Revision queue is full. Further > revisions will be compacted." is logged only when it *wasn't* blocking. > BUT, when queue empties, blocking flag is reset inside if block for > commitRateLimiter!=null. That means an event chain like: > # qFull > # log warn > # qEmpties > # qFull > won't log the WARN after step(4) > \[0]: http://markmail.org/message/hgein5g3ohyjhw5n > \[1]: > https://github.com/apache/jackrabbit-oak/blob/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/ChangeProcessor.java#L307 -- This message was sent by Atlassian JIRA (v6.3.15#6346)