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.
---