Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/5218#discussion_r159135944 --- 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 -- this is the same case as `"a|"` in `testTailingEmptyField()`. We only need one of these.
---