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 2022/03/25 05:55:39 UTC

[GitHub] [bookkeeper] eolivelli commented on a change in pull request #3103: add async or sync for :entry location index's rocksdb write

eolivelli commented on a change in pull request #3103:
URL: https://github.com/apache/bookkeeper/pull/3103#discussion_r834972991



##########
File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java
##########
@@ -1809,6 +1810,28 @@ public ServerConfiguration setIsForceGCAllowWhenNoSpace(boolean force) {
         return this;
     }
 
+    /**
+     * Get location index sync data.
+     *
+     * @return true  - sync operate location index,
+     *         false - async operate location index.
+     */
+    public boolean getLocationIndexSyncData() {

Review comment:
       This option is not for every LedgerStorage.
   We should reflect this into the name




-- 
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: issues-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org