hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Szehon Ho (JIRA)" <>
Subject [jira] [Updated] (HIVE-7730) Extend ReadEntity to add accessed columns from query
Date Thu, 28 Aug 2014 05:41:58 GMT


Szehon Ho updated HIVE-7730:

       Resolution: Fixed
    Fix Version/s: 0.14.0
           Status: Resolved  (was: Patch Available)

Committed to trunk.  Thanks Xiaomeng for the contribution!  [~brocknoland], [~ashutoshc] can
one of you please add Xiaomeng to list of contributors to accredit the patch?  Thanks.

> Extend ReadEntity to add accessed columns from query
> ----------------------------------------------------
>                 Key: HIVE-7730
>                 URL:
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Xiaomeng Huang
>             Fix For: 0.14.0
>         Attachments: HIVE-7730.001.patch, HIVE-7730.002.patch, HIVE-7730.003.patch, HIVE-7730.004.patch
> -Now what we get from HiveSemanticAnalyzerHookContextImpl is limited. If we have hook
of HiveSemanticAnalyzerHook, we may want to get more things from hookContext. (e.g. the needed
colums from query).-
> -So we should get instance of HiveSemanticAnalyzerHookContext from configuration, extends
HiveSemanticAnalyzerHookContext with a new implementation, overide the HiveSemanticAnalyzerHookContext.update()
and put what you want to the class.-
> Hive should store accessed columns to ReadEntity when we set HIVE_STATS_COLLECT_SCANCOLS(or
we can add a confVar) is true.
> Then external authorization model can get accessed columns when do authorization in compile
before execute. Maybe we will remove columnAccessInfo from BaseSemanticAnalyzer, old authorization
and AuthorizationModeV2 can get accessed columns from ReadEntity too.
> Here is the quick implement in SemanticAnalyzer.analyzeInternal() below:
> {code}   boolean isColumnInfoNeedForAuth = SessionState.get().isAuthorizationModeV2()
>         && HiveConf.getBoolVar(conf, HiveConf.ConfVars.HIVE_AUTHORIZATION_ENABLED);
>     if (isColumnInfoNeedForAuth
>         || HiveConf.getBoolVar(this.conf, HiveConf.ConfVars.HIVE_STATS_COLLECT_SCANCOLS)
== true) {
>       ColumnAccessAnalyzer columnAccessAnalyzer = new ColumnAccessAnalyzer(pCtx);
>       setColumnAccessInfo(columnAccessAnalyzer.analyzeColumnAccess()); 
>     }
>     compiler.compile(pCtx, rootTasks, inputs, outputs);
>     // TODO: 
>     // after compile, we can put accessed column list to ReadEntity getting from columnAccessInfo
> {code}

This message was sent by Atlassian JIRA

View raw message