Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/5218#discussion_r159670229
--- Diff:
flink-core/src/main/java/org/apache/flink/types/parser/StringParser.java ---
@@ -42,6 +42,12 @@ public void enableQuotedStringParsing(byte
quoteCharacter) {
@Override
public int parseField(byte[] bytes, int startPos, int limit, byte[]
delimiter, String reusable) {
+ if (startPos == limit) {
+ setErrorState(ParseErrorState.EMPTY_COLUMN);
+ this.result = new String(bytes, startPos, limit -
startPos, getCharset());
--- End diff --
change to `this.result = "";`
also this early out means that the later check `if (limit == startPos)` in
line 93 will never be true and can be removed.
---
