----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24587/#review50701 -----------------------------------------------------------
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java <https://reviews.apache.org/r/24587/#comment88571> Tiniest of nits, and it's not your fault: unnecessary comma in "(size-wise), keys" - Lefty Leverenz On Aug. 14, 2014, 11:41 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24587/ > ----------------------------------------------------------- > > (Updated Aug. 14, 2014, 11:41 p.m.) > > > Review request for hive and Gopal V. > > > Bugs: HIVE-7617 > https://issues.apache.org/jira/browse/HIVE-7617 > > > Repository: hive-git > > > Description > ------- > > See jira > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java af9e198 > ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java 3110b0a > > ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/BytesBytesMultiHashMap.java > 8b674ea > > ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinBytesTableContainer.java > 1f68f7b > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorHashKeyWrapper.java > 5f6df78 > > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorHashKeyWrapperBatch.java > 4ebf924 > > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorMapJoinOperator.java > 607f641 > > ql/src/test/org/apache/hadoop/hive/ql/exec/persistence/TestBytesBytesMultiHashMap.java > b3582b2 > serde/src/java/org/apache/hadoop/hive/serde2/ByteStream.java 390d9de > serde/src/java/org/apache/hadoop/hive/serde2/WriteBuffers.java bed4d0a > > serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java > 523ad7d > > Diff: https://reviews.apache.org/r/24587/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
