hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Teddy Choi" <>
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:

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

Review request for hive.


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.


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 for more details.

This addresses bug HIVE-3846.

Diffs (updated)

  ql/src/java/org/apache/hadoop/hive/ql/parse/ 48a05e8 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ 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 




Teddy Choi

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