Github user ggevay commented on a diff in the pull request:
https://github.com/apache/flink/pull/3511#discussion_r140663593
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/operators/sort/NormalizedKeySorter.java
---
@@ -309,7 +309,7 @@ public boolean write(T record) throws IOException {
// Access Utilities
//
------------------------------------------------------------------------
- private long readPointer(int logicalPosition) {
+ protected long readPointer(int logicalPosition) {
--- End diff --
Thanks, I've made them `protected final`:
db14ac58d137ed39f63bc4f8d724622f4cae518f
5e31cf011d5b2d3a9e986414411e312310867c44
---
