You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2018/07/06 00:08:51 UTC

[GitHub] reddycharan opened a new issue #1535: Bookie shutdown hook should trigger LedgerCacheImpl's indexPageManager flusher

reddycharan opened a new issue #1535: Bookie shutdown hook should trigger LedgerCacheImpl's indexPageManager flusher
URL: https://github.com/apache/bookkeeper/issues/1535
 
 
   
   **BUG REPORT**
   
   1. Please describe the issue you observed:
   
   As part of Bookie shutdown hook, InterleavedLedgerStorage.shutdown is called, which calls “entryLogger.shutdown()” and it flushes complete currentlogs and rotatedlogs.  But in InterleavedLedgerStorage.shutdown for ledgercache, it calls “ledgerCache.close()” and it just flushes FileInfos but not in memory LedgerEntryPage instances. To be in sync with EntryLog files, indexPageManager (memory pages) should also be flushed.
   
   https://github.com/apache/bookkeeper/blob/master/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/InterleavedLedgerStorage.java#L188
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services