You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Fredy Wijaya (Code Review)" <ge...@cloudera.org> on 2018/06/26 06:16:20 UTC
[Impala-ASF-CR] IMPALA-6988: Implement ALTER TABLE/VIEW SET OWNER
Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10822 )
Change subject: IMPALA-6988: Implement ALTER TABLE/VIEW SET OWNER
......................................................................
IMPALA-6988: Implement ALTER TABLE/VIEW SET OWNER
Alter the table/view owner to either user or role.
On table/view creation, the table/view owner will be set to
the current user, which can be viewed via DESCRIBE FORMATTED
command. Having an owner information allows implementing a
feature where an owner can be given certain privileges
automatically upon a table/view creation. See IMPALA-7075.
The ALTER TABLE/VIEW SET OWNER will be useful commands for
transferring ownership (a set of owner privileges) from the
current owner to another owner.
Syntax:
ALTER TABLE table SET OWNER USER user
ALTER TABLE table SET OWNER ROLE role
ALTER VIEW view SET OWNER USER user
ALTER VIEW view SET OWNER ROLE role
Testing:
- Added new FE tests
- Added new E2E tests
Change-Id: Ia1b75b1590b16eb0c2ba326d07ee3fd9897c27d1
---
M common/thrift/JniCatalog.thrift
M fe/src/main/cup/sql-parser.cup
A fe/src/main/java/org/apache/impala/analysis/AlterTableOrViewSetOwnerStmt.java
A fe/src/main/java/org/apache/impala/analysis/AlterTableSetOwnerStmt.java
M fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java
A fe/src/main/java/org/apache/impala/analysis/AlterViewSetOwnerStmt.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.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
12 files changed, 340 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/22/10822/5
--
To view, visit http://gerrit.cloudera.org:8080/10822
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia1b75b1590b16eb0c2ba326d07ee3fd9897c27d1
Gerrit-Change-Number: 10822
Gerrit-PatchSet: 5
Gerrit-Owner: Fredy Wijaya <fw...@cloudera.com>