[ 
https://issues.apache.org/jira/browse/FLINK-23237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17377059#comment-17377059
 ] 

hehuiyuan commented on FLINK-23237:
-----------------------------------

[~Aiden Gong] , Additionally, add log (debug or error) to print error data that 
failed to deserialize when set `ignore-parse-error` = `true`,  which is useful .

Is it considered to do?

> Add log to print data that failed to deserialize when  
> ignore-parse-error=true 
> -------------------------------------------------------------------------------
>
>                 Key: FLINK-23237
>                 URL: https://issues.apache.org/jira/browse/FLINK-23237
>             Project: Flink
>          Issue Type: Improvement
>          Components: Formats (JSON, Avro, Parquet, ORC, SequenceFile)
>            Reporter: hehuiyuan
>            Priority: Minor
>
> Add log to print error data that failed to deserialize when set 
> `ignore-parse-error` = `true`
>  
> {code:java}
> public RowData deserialize(@Nullable byte[] message) throws IOException {
>     if (message == null) {
>         return null;
>     }
>     try {
>         final JsonNode root = objectReader.readValue(message);
>         return (RowData) runtimeConverter.convert(root);
>     } catch (Throwable t) {
>         if (ignoreParseErrors) {
>             return null;
>         }
>         throw new IOException(
>                 String.format("Failed to deserialize CSV row '%s'.", new 
> String(message)), t);
>     }
> }
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to