Fang-Yu Rao has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/22843


Change subject: IMPALA-13982: [WIP] Support views for Calcite planner in Impala
......................................................................

IMPALA-13982: [WIP] Support views for Calcite planner in Impala

There are still some issues in this patch.

Specifically, the following could not be correctly handled even though
we already loaded the metadata of functional.alltypes_view to
'rootSchema' during populateCalciteSchema().

select functional.alltypes_view.id from functional.alltypes_view.

After we instantiate an instance of SqlBasicCall (v.s. SqlSelect) to
replace the SqlIdentifier that represents a view/Feview, the following
could be validated and executed. But the patch does not register the
privilege request for the column of 'test_v_01.id'.

select id from test_v_01;

select test_v_01.id from test_v_01;

On the other hand, we also need to register a privilege request for
'test_v_01' as a masked privilege request when this view was created
by a super user.

Change-Id: I6f7b1dea47022991c46879842f0471c55b297388
---
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/authorization/BaseAuthorizationChecker.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaHdfsScanRel.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/schema/CalciteTable.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisDriver.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteMetadataHandler.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/ImpalaPrivilegeRequestsRegistrar.java
A java/calcite-planner/src/main/java/org/apache/impala/calcite/sql/SqlView.java
8 files changed, 210 insertions(+), 29 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/43/22843/1
--
To view, visit http://gerrit.cloudera.org:8080/22843
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6f7b1dea47022991c46879842f0471c55b297388
Gerrit-Change-Number: 22843
Gerrit-PatchSet: 1
Gerrit-Owner: Fang-Yu Rao <[email protected]>

Reply via email to