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

Chunwei Lei commented on CALCITE-4098:
--------------------------------------

Attached the PR.

> Remove redundant code in "RelJson.toJson(RelDistribution)"
> ----------------------------------------------------------
>
>                 Key: CALCITE-4098
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4098
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Jiatao Tao
>            Assignee: Jiatao Tao
>            Priority: Minor
>
> RelDistribution#getKeys is annotated as nonnull, and must return 
> "List<Integer>", so this code is unnecessary.
>  
> {code:java}
> List<Object> keys = new ArrayList<>(relDistribution.getKeys().size()); 
>  for (Integer key : relDistribution.getKeys()) {
>   keys.add(toJson(key));
> }
>  
> {code}
>  
>  



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

Reply via email to