Github user sunjincheng121 commented on a diff in the pull request: https://github.com/apache/flink/pull/5218#discussion_r159139198 --- Diff: flink-core/src/test/java/org/apache/flink/types/parser/ParserTestBase.java --- @@ -407,26 +407,47 @@ public void testStaticParseMethodWithInvalidValues() { @Test public void testEmptyFieldInIsolation() { try { - String [] emptyStrings = new String[] {"|"}; + FieldParser<T> parser = getParser(); + + byte[] bytes = "|".getBytes(ConfigConstants.DEFAULT_CHARSET); --- End diff -- Yes, we can keep `testEmptyFieldInIsolation`.
---