[jira] [Resolved] (PARQUET-1723) Read From Maps Without Using Contains

["Fokko Driesprong (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/PARQUET-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Fokko Driesprong resolved PARQUET-1723.
---------------------------------------
    Fix Version/s: 1.12.0
       Resolution: Fixed

> Read From Maps Without Using Contains
> -------------------------------------
>
>                 Key: PARQUET-1723
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1723
>             Project: Parquet
>          Issue Type: Improvement
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.12.0
>
>
> I see a few places with the following pattern...
>  
> {code:java}
> if (map.contains(key)) {
>    return map.get(key);
> }
> {code}
> Better to just call {{get()}} and then check the return value for 'null' to determine if the key is there.  This prevents the need to traverse the {{Map}} twice,... once for {{contains}} and once for {{get}}.



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