You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by eo...@apache.org on 2018/01/18 01:49:08 UTC
[bookkeeper] branch master updated: revert DoubleByteBuf
niobuffers() changes
This is an automated email from the ASF dual-hosted git repository.
eolivelli 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 9fca590 revert DoubleByteBuf niobuffers() changes
9fca590 is described below
commit 9fca590d94bea9bf86a49f1dbdddf2ae37d3e949
Author: Sebastian Schepens <se...@mercadolibre.com>
AuthorDate: Thu Jan 18 02:48:58 2018 +0100
revert DoubleByteBuf niobuffers() changes
Revert some of the changes in https://github.com/apache/bookkeeper/pull/996 since they cause issues
Author: Sebastian Schepens <se...@mercadolibre.com>
Reviewers: Enrico Olivelli <eo...@gmail.com>, Sijie Guo <si...@apache.org>
This closes #998 from sschepens/double-bytebuf-fix-2
---
.../src/main/java/org/apache/bookkeeper/util/DoubleByteBuf.java | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/DoubleByteBuf.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/DoubleByteBuf.java
index 9c51601..4de6c0d 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/DoubleByteBuf.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/DoubleByteBuf.java
@@ -388,10 +388,7 @@ public final class DoubleByteBuf extends AbstractReferenceCountedByteBuf {
@Override
public ByteBuffer[] nioBuffers() {
- if (b1.nioBufferCount() == 1 && b2.nioBufferCount() == 1) {
- return new ByteBuffer[] { b1.nioBuffer(), b2.nioBuffer() };
- }
- return ObjectArrays.concat(b1.nioBuffers(), b2.nioBuffers(), ByteBuffer.class);
+ return nioBuffers(readerIndex(), readableBytes());
}
@Override
--
To stop receiving notification emails like this one, please contact
['"commits@bookkeeper.apache.org" <co...@bookkeeper.apache.org>'].