Fredy Wijaya has uploaded a new patch set (#7). ( 
http://gerrit.cloudera.org:8080/10471 )

Change subject: IMPALA-7016: Implement ALTER DATABASE SET OWNER
......................................................................

IMPALA-7016: Implement ALTER DATABASE SET OWNER

Alter the database owner to either user or role.

On database creation, the database owner will be set to the current
user, which can be viewed via DESCRIBE DATABASE db command. Having an
owner information allows implementing a feature where an owner can be
given certain privileges automatically upon a database creation. The
ALTER DATABASE SET OWNER will be a useful command for transferring
ownership (a set of owner privileges) from the current owner to another
owner.

Syntax:
ALTER DATABASE db SET OWNER USER user
ALTER DATABASE db SET OWNER ROLE role

Testing:
- Added new front-end tests
- Added new end-to-end tests

Change-Id: Ie3b923021ebce5192d2d64784e7ddb952ba82bc3
---
M common/thrift/CatalogService.thrift
M common/thrift/JniCatalog.thrift
M fe/src/main/cup/sql-parser.cup
A fe/src/main/java/org/apache/impala/analysis/AlterDbSetOwnerStmt.java
A fe/src/main/java/org/apache/impala/analysis/AlterDbStmt.java
M fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java
A fe/src/main/java/org/apache/impala/analysis/Owner.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M fe/src/main/java/org/apache/impala/util/MetaStoreUtil.java
M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java
M fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java
M fe/src/test/java/org/apache/impala/analysis/ParserTest.java
M tests/metadata/test_ddl.py
M tests/metadata/test_ddl_base.py
15 files changed, 361 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/10471/7
--
To view, visit http://gerrit.cloudera.org:8080/10471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie3b923021ebce5192d2d64784e7ddb952ba82bc3
Gerrit-Change-Number: 10471
Gerrit-PatchSet: 7
Gerrit-Owner: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Adam Holley <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>

Reply via email to