> 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 > >