----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37985/#review98251 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java (line 273) <https://reviews.apache.org/r/37985/#comment154571> Is this hbase-metastore branch patch or trunk patch? If its trunk then this change doesn't reflect the JIRA title. ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java (line 1397) <https://reviews.apache.org/r/37985/#comment154577> Same reasoning again. If trunk patch, please create a followup jira and include this change as it is not used anywhere and is not a related change. - Prasanth_J On Sept. 8, 2015, 11:09 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37985/ > ----------------------------------------------------------- > > (Updated Sept. 8, 2015, 11:09 p.m.) > > > Review request for hive and Prasanth_J. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java cf8694e > ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcSerde.java 8beff4b > ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java fcb3746 > ql/src/java/org/apache/hadoop/hive/ql/io/parquet/ProjectionPusher.java > 4480600 > ql/src/java/org/apache/hadoop/hive/ql/io/sarg/ConvertAstToSearchArg.java > e034650 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/ColumnPrunerProcFactory.java > 2dc15f9 > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java f6052e3 > serde/src/java/org/apache/hadoop/hive/serde2/ColumnProjectionUtils.java > 10086c5 > > storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgumentFactory.java > 0778935 > > storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgumentImpl.java > d27ac16 > > Diff: https://reviews.apache.org/r/37985/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
