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

Posted to issues-all@impala.apache.org by "Zoltán Borók-Nagy (Jira)" <ji...@apache.org> on 2021/11/19 16:29:00 UTC

[jira] [Created] (IMPALA-11031) Listmap.getIndex() name is misleading

Zoltán Borók-Nagy created IMPALA-11031:
------------------------------------------

             Summary: Listmap.getIndex() name is misleading
                 Key: IMPALA-11031
                 URL: https://issues.apache.org/jira/browse/IMPALA-11031
             Project: IMPALA
          Issue Type: Improvement
          Components: Frontend
            Reporter: Zoltán Borók-Nagy


Listmap.getIndex(t) modifies the ListMap object when there is no mapping for 't'.

Hence the name of it is very misleading as the reader wouldn't expect modifications from simple getters.

Probably we should just rename it to getOrAddIndex().



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org