Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/5218#discussion_r159667871
--- Diff:
flink-core/src/main/java/org/apache/flink/types/parser/LongValueParser.java ---
@@ -89,6 +95,11 @@ public int parseField(byte[] bytes, int startPos, int
limit, byte[] delimiter, L
}
}
+ if (startPos == limit) {
--- End diff --
Why do we need this additional check?
This should never evaluate to `true` because the condition is checked
before.
---
