Github user pnowojski commented on a diff in the pull request:
https://github.com/apache/flink/pull/6071#discussion_r194711763
--- Diff:
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/network/ChunkedByteBuf.java
---
@@ -88,6 +102,16 @@ public ByteBuf readChunk(ChannelHandlerContext ctx)
throws Exception {
}
}
+ @Override
+ public long length() {
+ return length;
--- End diff --
It seems like content of `buf` is not changing. However we can not guard
against it programmatically, only via java doc. On the other hand, I'm not sure
what would the side effects of "unknown" length be.
---
