[improve] [bookie] Enable reorder read sequence for bk client by defa…

…ult (#21652) Co-authored-by: Jiwe Guo <[email protected]>
apache · Dec 4, 2023 · 0300019 · 0300019
1 parent aa3fcc9
commit 0300019
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/conf/bookkeeper.conf b/conf/bookkeeper.conf
@@ -255,6 +255,9 @@ rereplicationEntryBatchSize=100
 # Enable/disable having read operations for a ledger to be sticky to a single bookie.
 stickyReadSEnabled=true
 
+# Enable/disable reordering read sequence on reading entries.
+reorderReadSequenceEnabled=true
+
 # Auto-replication
 # The grace period, in milliseconds, that the replication worker waits before fencing and
 # replicating a ledger fragment that's still being written to upon bookie failure.

diff --git a/conf/broker.conf b/conf/broker.conf
@@ -1015,7 +1015,7 @@ bookkeeperClientMinNumRacksPerWriteQuorum=2
 bookkeeperClientEnforceMinNumRacksPerWriteQuorum=false
 
 # Enable/disable reordering read sequence on reading entries.
-bookkeeperClientReorderReadSequenceEnabled=false
+bookkeeperClientReorderReadSequenceEnabled=true
 
 # Enable bookie isolation by specifying a list of bookie groups to choose from. Any bookie
 # outside the specified groups will not be used by the broker

diff --git a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
@@ -1757,7 +1757,7 @@ The delayed message index time step(in seconds) in per bucket snapshot segment,
     @FieldContext(
         category = CATEGORY_STORAGE_BK,
         doc = "Enable/disable reordering read sequence on reading entries")
-    private boolean bookkeeperClientReorderReadSequenceEnabled = false;
+    private boolean bookkeeperClientReorderReadSequenceEnabled = true;
     @FieldContext(
         category = CATEGORY_STORAGE_BK,
         required = false,