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 <[email protected]>
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 <[email protected]>
Reviewers: Enrico Olivelli <[email protected]>, Sijie Guo
<[email protected]>
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
['"[email protected]" <[email protected]>'].
