-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63845/#review191240
-----------------------------------------------------------




ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
Lines 1319 (patched)
<https://reviews.apache.org/r/63845/#comment268936>

    Does this assume database is current?
    If so, thats fine but include comment in code and in wiki about syntax.



ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java
Lines 4081 (patched)
<https://reviews.apache.org/r/63845/#comment268937>

    current db?



ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
Lines 266 (patched)
<https://reviews.apache.org/r/63845/#comment268939>

    nit: ws


- Ashutosh Chauhan


On Nov. 15, 2017, 5:52 p.m., Jesús Camacho Rodríguez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63845/
> -----------------------------------------------------------
> 
> (Updated Nov. 15, 2017, 5:52 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Bugs: HIVE-15018
>     https://issues.apache.org/jira/browse/HIVE-15018
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-15018
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 
> 54491f41e2588db7e7a64fe178ce47835cc19941 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 
> 7961b2b912f03b6c72ca60b06d7eda5026b691ae 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 
> f5c162d80a6c521715e2d296c38924539e485594 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java 
> 64d16750afabbe1a8ba39ab6897c3515f3e82753 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/AlterMaterializedViewDesc.java 
> PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/DDLWork.java 
> 369f8440a25c2329f307d1fa5875006b3bbb8f41 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/HiveOperation.java 
> 61c540789791cd09d4f074193b09e94c9cb4ba65 
>   
> ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveOperationType.java
>  4be42c1935ecbead08c49e0bf234f4f196b6bb7a 
>   
> ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
>  080880be6e9e5fde81e5c3f27aac8431a8818838 
>   ql/src/test/queries/clientpositive/materialized_view_create_rewrite.q 
> b17517f76be6809bdb9733248a87825745319775 
>   
> ql/src/test/results/clientpositive/beeline/materialized_view_create_rewrite.q.out
>  f6b161b690880a554c13aa02682b461530fa7686 
>   ql/src/test/results/clientpositive/materialized_view_create_rewrite.q.out 
> f6b161b690880a554c13aa02682b461530fa7686 
> 
> 
> Diff: https://reviews.apache.org/r/63845/diff/2/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jesús Camacho Rodríguez
> 
>

Reply via email to