----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51735/#review148587 -----------------------------------------------------------
Ship it! Ship It! - nabarun nag On Sept. 8, 2016, 10:26 p.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51735/ > ----------------------------------------------------------- > > (Updated Sept. 8, 2016, 10:26 p.m.) > > > Review request for geode, Barry Oglesby and nabarun nag. > > > Repository: geode > > > Description > ------- > > When deciding whether to combine the predicates into a common junction, the > type should be checked to make sure they are the same. This will prevent OR > junctions from being added/converted to an AND junction and vice versa. This > was only affecting certain query paths. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/cache/query/internal/AbstractGroupOrRangeJunction.java > a14dda8 > > geode-core/src/test/java/com/gemstone/gemfire/cache/query/dunit/CompactRangeIndexQueryDUnitTest.java > 8b27309 > > Diff: https://reviews.apache.org/r/51735/diff/ > > > Testing > ------- > > > Thanks, > > Jason Huynh > >