You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@calcite.apache.org by "James Starr (Jira)" <ji...@apache.org> on 2020/09/25 23:34:00 UTC

[jira] [Commented] (CALCITE-4280) Replace Guava's Lists.transform with Util.transform

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

James Starr commented on CALCITE-4280:
--------------------------------------

I agree that is should be stuck on the forbidden list, but, why not use java.util.Stream?

> Replace Guava's Lists.transform with Util.transform
> ---------------------------------------------------
>
>                 Key: CALCITE-4280
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4280
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.25.0
>         Environment: Lists.transform has wrong generic signature, and it relies on Guava's Function types.
> We should stick to a single method, so I suggest we go with Util.transform, and we should probably add Lists.transform to forbidden-apis list.
>            Reporter: Vladimir Sitnikov
>            Assignee: Vladimir Sitnikov
>            Priority: Major
>




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