Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16529
to look at the new patch set (#4).
Change subject: IMPALA-10172: Support Hive metastore managed locations for
databases
......................................................................
IMPALA-10172: Support Hive metastore managed locations for databases
This property sets the location where the database's tables with
table property 'transactional'='true' will be placed.
The change also adds managedlocation to DESCRIBE DATABASE's output.
Note that Impala's output for DESCRIBE DATABASE (EXTENDED) is
different than Hive's. Change to Hive's format would be preferable
in my opinion, but is a potentially braking change. See IMPALA-6686
for further discussion.
Testing:
- added FE and EE tests
- ran relevant tests
Change-Id: I925632a43ff224f762031e89981896722e453399
---
M common/thrift/JniCatalog.thrift
M fe/src/main/cup/sql-parser.cup
M fe/src/main/java/org/apache/impala/analysis/CreateDbStmt.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M fe/src/main/java/org/apache/impala/service/DescribeResultFactory.java
M fe/src/main/jflex/sql-scanner.flex
M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java
M fe/src/test/java/org/apache/impala/authorization/AuthorizationStmtTest.java
M testdata/workloads/functional-query/queries/QueryTest/create-database.test
M testdata/workloads/functional-query/queries/QueryTest/describe-db.test
M testdata/workloads/functional-query/queries/QueryTest/describe-hive-db.test
M tests/metadata/test_metadata_query_statements.py
12 files changed, 197 insertions(+), 9 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/29/16529/4
--
To view, visit http://gerrit.cloudera.org:8080/16529
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I925632a43ff224f762031e89981896722e453399
Gerrit-Change-Number: 16529
Gerrit-PatchSet: 4
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
