hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Teddy Choi" <pudi...@gmail.com>
Subject Re: Review Request: HIVE-3846 alter view rename NPEs with authorization on.
Date Wed, 08 May 2013 01:33:17 GMT

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

(Updated May 8, 2013, 1:33 a.m.)


Review request for hive.


Changes
-------

Summary of changes
- authorization_8.q.out is included.
- authorization_8.q does not make other tests fail.
- alter_view_rename.q.out and recursive_view.q.out is updated.


Description
-------

HiveOperation.ALTERVIEW_RENAME was missing.
I added it on HiveOperation and SemanticAnalyzerFactory. Also I added a test to reproduce
the error as Ashutosh Chauhan mentioned and it was fixed.

Please read https://issues.apache.org/jira/browse/HIVE-3846 for more details.


This addresses bug HIVE-3846.
    https://issues.apache.org/jira/browse/HIVE-3846


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java 48a05e8 
  ql/src/java/org/apache/hadoop/hive/ql/plan/HiveOperation.java 4b1c429 
  ql/src/test/queries/clientpositive/authorization_8.q PRE-CREATION 
  ql/src/test/results/clientnegative/recursive_view.q.out 3a2618b 
  ql/src/test/results/clientpositive/alter_view_rename.q.out 6a498e8 
  ql/src/test/results/clientpositive/authorization_8.q.out PRE-CREATION 

Diff: https://reviews.apache.org/r/10771/diff/


Testing
-------


Thanks,

Teddy Choi


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message