You are viewing a plain text version of this content. The canonical link for it is here.
Posted to distributedlog-commits@bookkeeper.apache.org by si...@apache.org on 2017/10/04 08:48:25 UTC

[distributedlog] branch master updated: ISSUE #181: Enable LogRecord ByteBuf based constructor public

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 9de4eb5  ISSUE #181: Enable LogRecord ByteBuf based constructor public
9de4eb5 is described below

commit 9de4eb502c1c09cc84c84ce1093faae3bbdf406c
Author: Arvin <ar...@gmail.com>
AuthorDate: Wed Oct 4 01:48:15 2017 -0700

    ISSUE #181: Enable LogRecord ByteBuf based constructor public
    
    Descriptions of the changes in this PR:
    change LogRecord ByteBuf based constructor to public
    
    Author: Arvin <ar...@gmail.com>
    
    Reviewers: Sijie Guo <si...@apache.org>
    
    This closes #182 from ArvinDevel/issue_181, closes #181
---
 .../src/main/java/org/apache/distributedlog/LogRecord.java           | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/distributedlog-protocol/src/main/java/org/apache/distributedlog/LogRecord.java b/distributedlog-protocol/src/main/java/org/apache/distributedlog/LogRecord.java
index 7a199f6..5d666a0 100644
--- a/distributedlog-protocol/src/main/java/org/apache/distributedlog/LogRecord.java
+++ b/distributedlog-protocol/src/main/java/org/apache/distributedlog/LogRecord.java
@@ -171,12 +171,11 @@ public class LogRecord {
     }
 
     /**
-     * Used by {@link LogRecordWithDLSN} to construct a log record read by readers.
-     *
+     * Construct a log record with <i>txid</i> and ByteBuf <i>payload</i>.
      * @param txid transaction id
      * @param payload playload
      */
-    protected LogRecord(long txid, ByteBuf payload) {
+    public LogRecord(long txid, ByteBuf payload) {
         this.txid = txid;
         // Need to make a copy since the passed payload is using a ref-count buffer that we don't know when could
         // release, since the record is passed to the user. Also, the passed ByteBuf is coming from network and is

-- 
To stop receiving notification emails like this one, please contact
['"distributedlog-commits@bookkeeper.apache.org" <di...@bookkeeper.apache.org>'].