You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by iv...@apache.org on 2018/01/26 17:19:10 UTC

[bookkeeper] branch master updated: [MERGE YAHOO REPO] Release addEntry-Bytebuf on readOnlyBookie to prevent memory-leak

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

ivank 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 ffbe842  [MERGE YAHOO REPO] Release addEntry-Bytebuf on readOnlyBookie to prevent memory-leak
ffbe842 is described below

commit ffbe842d11ddd4e64ee79a3b6f86183be942cfc4
Author: rdhabalia <rd...@yahoo-inc.com>
AuthorDate: Fri Jan 26 18:19:03 2018 +0100

    [MERGE YAHOO REPO] Release addEntry-Bytebuf on readOnlyBookie to prevent memory-leak
    
    Descriptions of the changes in this PR:
    This is cherry-pick from yahoo repo of branch yahoo-4.3.
    
    original commit is:
    https://github.com/yahoo/bookkeeper/commit/42bdc083
    Release addEntry-Bytebuf on readOnlyBookie to prevent memory-leak
    
    Author: rdhabalia <rd...@yahoo-inc.com>
    
    Reviewers: Ivan Kelly <iv...@apache.org>
    
    This closes #1059 from jiazhai/cherry_picks/i_173
---
 .../src/main/java/org/apache/bookkeeper/proto/WriteEntryProcessor.java   | 1 +
 1 file changed, 1 insertion(+)

diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/WriteEntryProcessor.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/WriteEntryProcessor.java
index 87df2d2..db2cd5e 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/WriteEntryProcessor.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/WriteEntryProcessor.java
@@ -66,6 +66,7 @@ class WriteEntryProcessor extends PacketProcessorBase implements WriteCallback {
                          ResponseBuilder.buildErrorResponse(BookieProtocol.EREADONLY, add),
                          requestProcessor.addRequestStats);
             add.release();
+            add.recycle();
             return;
         }
 

-- 
To stop receiving notification emails like this one, please contact
ivank@apache.org.