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

(Updated Aug. 2, 2014, 1:49 a.m.)


Review request for hive and Thejas Nair.


Changes
-------

Updates per review feedback from Thejas


Bugs: HIVE-7583
    https://issues.apache.org/jira/browse/HIVE-7583


Repository: hive-git


Description
-------

Create shims method to either use new FileSystem.access() API, or fall back to 
Hive implementation of file access checks.


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/FileUtils.java 8dcd4cf 
  
itests/hive-unit-hadoop2/src/test/java/org/apache/hadoop/hive/ql/security/TestStorageBasedMetastoreAuthorizationProviderWithACL.java
 PRE-CREATION 
  
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/TestMetastoreAuthorizationProvider.java
 97fb7ba 
  
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/TestStorageBasedMetastoreAuthorizationProvider.java
 223f155 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/StorageBasedAuthorizationProvider.java
 f803cc4 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLAuthorizationUtils.java
 6a283ab 
  shims/0.20/src/main/java/org/apache/hadoop/hive/shims/Hadoop20Shims.java 
d03f270 
  shims/0.23/src/main/java/org/apache/hadoop/hive/shims/Hadoop23Shims.java 
e98b54d 
  
shims/common-secure/src/main/java/org/apache/hadoop/hive/shims/HadoopShimsSecure.java
 605ea55 
  shims/common/src/main/java/org/apache/hadoop/fs/DefaultFileAccess.java 
PRE-CREATION 
  shims/common/src/main/java/org/apache/hadoop/hive/shims/HadoopShims.java 
eefd5e5 

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


Testing
-------


Thanks,

Jason Dere

Reply via email to