doris-flink-connector
doris-flink-connector copied to clipboard
[Fix] fix deserialization error for QueryPlan exception
Proposed changes
Issue Number: close #65
Problem Summary:
Deserialization error on QueryPlan with exceptions.
Checklist(Required)
- Does it affect the original behavior: (No)
- Has unit tests been added: (No Need)
- Has document been added or modified: (No Need)
- Does it need to update dependencies: (No)
- Are there any changes that cannot be rolled back: (No)
Further comments
If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...
Thank you for your pr, is it also possible to add @JsonIgnoreProperties(ignoreUnknown = true)
directly to the entity class of QueryPlan
?
Yes. Do you mean adding the annotation alone or also with the exception
field? I think the exception msg is helpful and should be exposed to users properly.
However, the response of queryPlan has been judged in RestService.getQueryPlan
, and an exception
is also thrown when an error occurs, so is there no need to record the exception in queryPlan?