> On March 22, 2018, 12:26 a.m., Jesús Camacho Rodríguez wrote: > > storage-api/src/java/org/apache/hive/common/util/TxnIdUtils.java > > Lines 41 (patched) > > <https://reviews.apache.org/r/66204/diff/1/?file=1985166#file1985166line41> > > > > Can minOpenId be null for both lists? What's the semantics then?
Ok, looking at TxnUtils.createValidReaderWriteIdList(), it looks like it could potentially be null in the case that there is no min open ID (they are all committed?). Eugene does that sound right? In which case, if both are in that condition then I suspect they would be at the same commit point only if they have the same highwater mark (if they have different highwater marks then one of them has more committed IDs than the other). I'll update the logic and tests. - Jason ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66204/#review199731 ----------------------------------------------------------- On March 22, 2018, 12:16 a.m., Jason Dere wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66204/ > ----------------------------------------------------------- > > (Updated March 22, 2018, 12:16 a.m.) > > > Review request for hive, Eugene Koifman, Gopal V, and Jesús Camacho Rodríguez. > > > Bugs: HIVE-19017 > https://issues.apache.org/jira/browse/HIVE-19017 > > > Repository: hive-git > > > Description > ------- > > Create utility comparison function and some simple tests. > > > Diffs > ----- > > storage-api/src/java/org/apache/hive/common/util/TxnIdUtils.java > PRE-CREATION > storage-api/src/test/org/apache/hive/common/util/TestTxnIdUtils.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/66204/diff/1/ > > > Testing > ------- > > > Thanks, > > Jason Dere > >
