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.
      


---

Reply via email to