Hello Tianyi Wang, Vuk Ercegovac,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/10798
to review the following change.
Change subject: IMPALA-7140 (part 7): small fixes to enable most queries on
HDFS tables
......................................................................
IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables
This is a grab-bag of small fixes necessary to get most queries on HDFS
tables passing with the correct plans:
* Change the loading of tables to check for other table types before
checking for FS tables, since without specific checks against
various properties, other tables may look like FS tables and try to
instantiate LocalFsTable incorrectly.
* Return -1 for extrapolated row count -- the previous 0 value was
convincing the planner that it had a valid value.
* Fix the "builtins DB" code path. This is currently quite messy since
some planner classes refer to ImpaladCatalog directly to get a
BuiltinsDb, and so we need to use that instance rather than creating
our own for now.
* Properly handle the case where all partitions are pruned by a
predicate.
With this change, about half of the tests in PlannerTest pass. The tests
that don't pass all rely on views, HBase tables, etc.
Change-Id: I6f603e62b7a013c148c0905ebdec2f4303f9c4e5
---
M fe/src/main/java/org/apache/impala/catalog/local/LocalCatalog.java
M fe/src/main/java/org/apache/impala/catalog/local/LocalFsTable.java
M fe/src/main/java/org/apache/impala/catalog/local/LocalTable.java
M fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java
4 files changed, 33 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/10798/1
--
To view, visit http://gerrit.cloudera.org:8080/10798
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6f603e62b7a013c148c0905ebdec2f4303f9c4e5
Gerrit-Change-Number: 10798
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Tianyi Wang <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
