Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/5423#discussion_r167621192
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/buffer/Buffer.java
---
@@ -60,12 +60,20 @@
void tagAsEvent();
/**
- * Returns the underlying memory segment.
+ * Returns the underlying memory segment. This method is dangerous
since it ignores read only protections and omits
+ * slices. Use it only along the {@link #getMemorySegmentOffset()}.
*
* @return the memory segment backing this buffer
*/
+ @Deprecated
MemorySegment getMemorySegment();
+ /**
+ * @return the offset where this (potential slice) {@link Buffer}'s
data start in the underlying memory segment.
+ */
+ @Deprecated
--- End diff --
Same here.
---