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




agents-audit/src/main/java/org/apache/ranger/audit/provider/AuditFileCacheProvider.java
Line 62 (original), 62 (patched)
<https://reviews.apache.org/r/63552/#comment268380>

    Is this a bug regardless of the ORC change?



agents-audit/src/main/java/org/apache/ranger/audit/utils/AuditWriter.java
Lines 130 (patched)
<https://reviews.apache.org/r/63552/#comment268381>

    Will this pick up configs from /etc/hadoop/conf by default? This will add 
audit configs?



agents-audit/src/main/java/org/apache/ranger/audit/utils/AuditWriter.java
Lines 145 (patched)
<https://reviews.apache.org/r/63552/#comment268382>

    Will these folders have the correct permissions?



agents-audit/src/main/java/org/apache/ranger/audit/utils/AuditWriter.java
Lines 162 (patched)
<https://reviews.apache.org/r/63552/#comment268383>

    Is this necessary? Can't this be overridden by the writer itself? Like ORC 
writer would have orc extension. A potential Avro writer would have avro 
extension. Without having to modify this class.



agents-audit/src/main/java/org/apache/ranger/audit/utils/ORCFileUtil.java
Lines 96 (patched)
<https://reviews.apache.org/r/63552/#comment268384>

    Agree with this. It would be nice if this was passed down to the writer.



agents-audit/src/main/java/org/apache/ranger/audit/utils/ORCWriter.java
Lines 183 (patched)
<https://reviews.apache.org/r/63552/#comment268385>

    This logic should be in the ORCFileUtil?


- Kevin Risden


On Nov. 3, 2017, 7:38 p.m., Ramesh Mani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63552/
> -----------------------------------------------------------
> 
> (Updated Nov. 3, 2017, 7:38 p.m.)
> 
> 
> Review request for ranger, Don Bosco Durai and Madhan Neethiraj.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> RANGER-1837:Enhance Ranger Audit to HDFS to support ORC file format
> 
> 
> Diffs
> -----
> 
>   agents-audit/README.txt PRE-CREATION 
>   agents-audit/pom.xml c8bd1d8 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/destination/HDFSAuditDestination.java
>  66d8504 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/provider/AuditFileCacheProvider.java
>  314b130 
>   agents-audit/src/main/java/org/apache/ranger/audit/provider/MiscUtil.java 
> eff3824 
>   agents-audit/src/main/java/org/apache/ranger/audit/utils/AuditWriter.java 
> PRE-CREATION 
>   agents-audit/src/main/java/org/apache/ranger/audit/utils/ORCFileUtil.java 
> PRE-CREATION 
>   agents-audit/src/main/java/org/apache/ranger/audit/utils/ORCWriter.java 
> PRE-CREATION 
>   agents-audit/src/main/java/org/apache/ranger/audit/utils/TextWriter.java 
> PRE-CREATION 
>   agents-audit/src/main/java/org/apache/ranger/audit/utils/Writer.java 
> PRE-CREATION 
>   pom.xml 589cd6a 
>   src/main/assembly/hbase-agent.xml 3ebc334 
>   src/main/assembly/hdfs-agent.xml 5279a9a 
>   src/main/assembly/hive-agent.xml ca65c80 
>   src/main/assembly/knox-agent.xml 8357d49 
>   src/main/assembly/plugin-atlas.xml fd98811 
>   src/main/assembly/plugin-kafka.xml 95855d9 
>   src/main/assembly/plugin-kms.xml 6d15f2a 
>   src/main/assembly/plugin-solr.xml de30bfb 
>   src/main/assembly/plugin-sqoop.xml d2bd69a 
>   src/main/assembly/plugin-yarn.xml c6a48e8 
>   src/main/assembly/storm-agent.xml 64224ec 
> 
> 
> Diff: https://reviews.apache.org/r/63552/diff/3/
> 
> 
> Testing
> -------
> 
> Testing done in local
> 
> 
> Thanks,
> 
> Ramesh Mani
> 
>

Reply via email to