Impala Public Jenkins has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/8018 Reviewed-by: Lars Volker <[email protected]> Reviewed-by: Alex Behm <[email protected]> Tested-by: Impala Public Jenkins --- M fe/src/main/java/org/apache/impala/catalog/HBaseTable.java 1 file changed, 8 insertions(+), 8 deletions(-) Approvals: Impala Public Jenkins: Verified Lars Volker: Looks good to me, but someone else must approve Alex Behm: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8018 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35f0a3cd9363b55b3cde237beaf3037a7967582a Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tianyi Wang <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker <[email protected]>
