Github user sunjincheng121 commented on a diff in the pull request:
https://github.com/apache/flink/pull/5218#discussion_r159682488
--- 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 --
I find new String will do some check.. but change to `this.result=""` also
make sense.
---
