> On Aug. 14, 2015, 9:15 p.m., Prasanth_J wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/io/HdfsUtils.java, line 31
> > <https://reviews.apache.org/r/37425/diff/1/?file=1038978#file1038978line31>
> >
> >     I don't see this class being used anywhere. Remove it?

it will be used by future patches :)


- Sergey


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


On Aug. 13, 2015, 12:35 a.m., Sergey Shelukhin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37425/
> -----------------------------------------------------------
> 
> (Updated Aug. 13, 2015, 12:35 a.m.)
> 
> 
> Review request for hive, Gopal V and Prasanth_J.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 9cc7987 
>   ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java c7e0780 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HdfsUtils.java PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java 4e6dd7a 
>   ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcSplit.java 8cf4cc0 
>   ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java 
> 6c77ba4 
>   ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Initiator.java 32a9ef8 
>   ql/src/test/org/apache/hadoop/hive/ql/io/TestAcidUtils.java f8ded12 
>   ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java 
> 6cb8529 
>   shims/0.20S/src/main/java/org/apache/hadoop/hive/shims/Hadoop20SShims.java 
> ffffcb7 
>   shims/0.23/src/main/java/org/apache/hadoop/hive/shims/Hadoop23Shims.java 
> 9eae0ac 
>   shims/common/src/main/java/org/apache/hadoop/hive/shims/HadoopShims.java 
> 74785e5 
> 
> Diff: https://reviews.apache.org/r/37425/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Shelukhin
> 
>

Reply via email to