impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tianyi Wang (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4082: Remove todo item in getRegionsInRange
Date Sat, 09 Sep 2017 00:27:20 GMT
Tianyi Wang has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/8018

Change subject: IMPALA-4082: Remove todo item in getRegionsInRange
......................................................................

IMPALA-4082: Remove todo item in getRegionsInRange

HbaseTable.GetRegionsInRange is a function copied and modified from
HTable in Hbase 0.95.1 or ealier. The HTable function uses cached region
location and the modified version gets more up-to-date information.
There is a todo item for the removal of this modified function if
Hbase provides the same functionality itself. In Hbase 0.95.2, the
HTable function is renamed to getKeysAndRegionsInRange and in Hbase
0.99 it became a private function. Thus this todo item is no longer
needed and is removed by this patch.

Change-Id: I35f0a3cd9363b55b3cde237beaf3037a7967582a
---
M fe/src/main/java/org/apache/impala/catalog/HBaseTable.java
1 file changed, 8 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/18/8018/1
-- 
To view, visit http://gerrit.cloudera.org:8080/8018
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I35f0a3cd9363b55b3cde237beaf3037a7967582a
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tianyi Wang <twang@cloudera.com>

Mime
View raw message