Zoltan Borok-Nagy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18042
Change subject: IMPALA-11031: Listmap.getIndex() name is misleading ...................................................................... IMPALA-11031: Listmap.getIndex() name is misleading 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. This patch renames it to getOrAddIndex(). Change-Id: I689dfb67e1a9104812489d6299ed43446d2fcae8 --- M fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java M fe/src/main/java/org/apache/impala/catalog/HdfsPartitionLocationCompressor.java M fe/src/main/java/org/apache/impala/planner/DataSourceScanNode.java M fe/src/main/java/org/apache/impala/planner/HBaseScanNode.java M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java M fe/src/main/java/org/apache/impala/planner/KuduScanNode.java M fe/src/main/java/org/apache/impala/util/ListMap.java 7 files changed, 11 insertions(+), 10 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/18042/1 -- To view, visit http://gerrit.cloudera.org:8080/18042 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I689dfb67e1a9104812489d6299ed43446d2fcae8 Gerrit-Change-Number: 18042 Gerrit-PatchSet: 1 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
