This is an automated email from the ASF dual-hosted git repository.

lushiji pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new cec8267682 recycle LongPairWrapper (#3728)
cec8267682 is described below

commit cec8267682202dfb77342db4366fef07ad7632d6
Author: wenbingshen <[email protected]>
AuthorDate: Fri Feb 10 17:26:34 2023 +0800

    recycle LongPairWrapper (#3728)
---
 .../bookkeeper/bookie/storage/ldb/LocationsIndexRebuildOp.java    | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/LocationsIndexRebuildOp.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/LocationsIndexRebuildOp.java
index e0edc034cb..b9aaebec9e 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/LocationsIndexRebuildOp.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/storage/ldb/LocationsIndexRebuildOp.java
@@ -114,7 +114,13 @@ public class LocationsIndexRebuildOp {
                         // Update the ledger index page
                         LongPairWrapper key = LongPairWrapper.get(ledgerId, 
entryId);
                         LongWrapper value = LongWrapper.get(location);
-                        batch.get().put(key.array, value.array);
+
+                        try {
+                            batch.get().put(key.array, value.array);
+                        } finally {
+                            key.recycle();
+                            value.recycle();
+                        }
 
                         if (count.incrementAndGet() > BATCH_COMMIT_SIZE) {
                             batch.get().flush();

Reply via email to