Hello Vuk Ercegovac,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#6).

Change subject: IMPALA-7140 (part 1). Support fetching schema info in 
LocalCatalog
......................................................................

IMPALA-7140 (part 1). Support fetching schema info in LocalCatalog

This adds support for loading the Table object from HMS and parsing out
the Column and partitioning information from it.

With this change, I'm able to connect to an impalad running in "local
catalog" mode and run DESCRIBE, DESCRIBE EXTENDED, and SHOW CREATE TABLE
commands. Other commands like SHOW PARTITIONS don't work properly yet,
and type-specific table functionality (eg views, HBase tables, etc) are
not yet supported.

Again a simple unit test is included to check that column information is
loaded. More thorough testing is deferred until we've reached enough
coverage that we can start running e2e tests against a cluster running
in "local" mode.

Change-Id: I640f27e36198955e057da62a3ce25a858406e496
---
A fe/src/main/java/org/apache/impala/catalog/FeCatalogUtils.java
M fe/src/main/java/org/apache/impala/catalog/FeFsTable.java
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
M fe/src/main/java/org/apache/impala/catalog/Table.java
M fe/src/main/java/org/apache/impala/catalog/local/LocalCatalogException.java
M fe/src/main/java/org/apache/impala/catalog/local/LocalDb.java
M fe/src/main/java/org/apache/impala/catalog/local/LocalTable.java
M fe/src/test/java/org/apache/impala/catalog/CatalogTest.java
M fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java
9 files changed, 300 insertions(+), 79 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/30/10630/6
--
To view, visit http://gerrit.cloudera.org:8080/10630
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I640f27e36198955e057da62a3ce25a858406e496
Gerrit-Change-Number: 10630
Gerrit-PatchSet: 6
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>

Reply via email to