hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashutosh Chauhan <hashut...@apache.org>
Subject Re: Review Request 47916: HIVE-13857 insert overwrite select from some table fails throwing org.apache.hadoop.security.AccessControlException - II
Date Fri, 27 May 2016 21:20:36 GMT

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




ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java (lines 2867 - 2876)
<https://reviews.apache.org/r/47916/#comment200349>

    I don't think this is needed. This codepath gets exercised when dest is pre-existing.
There is no need to set permissions in that case.


- Ashutosh Chauhan


On May 27, 2016, 7:42 p.m., Hari Sankar Sivarama Subramaniyan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47916/
> -----------------------------------------------------------
> 
> (Updated May 27, 2016, 7:42 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> insert overwrite select from some table fails throwing org.apache.hadoop.security.AccessControlException
- II
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 717589a 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 3fa1233 
>   shims/common/src/main/java/org/apache/hadoop/hive/io/HdfsUtils.java c2060fc 
> 
> Diff: https://reviews.apache.org/r/47916/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Hari Sankar Sivarama Subramaniyan
> 
>


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