----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40415/#review116818 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java (line 675) <https://reviews.apache.org/r/40415/#comment177922> There's already an in-out parameter for results, so I feel it will make it unobvious if these are inconsistent - Sergey Shelukhin On Nov. 18, 2015, 12:23 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40415/ > ----------------------------------------------------------- > > (Updated Nov. 18, 2015, 12:23 a.m.) > > > Review request for hive, Gopal V, Prasanth_J, and Vikram Dixit Kumaraswamy. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 953e52c > > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > c5e7a5f > metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java > aa96f77 > ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java 46862da > ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 488d923 > ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java > ec90481 > storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/PredicateLeaf.java > dc71db4 > storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgument.java > d70b3b0 > > storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgumentImpl.java > eeff131 > > Diff: https://reviews.apache.org/r/40415/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
