----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36477/#review92964 -----------------------------------------------------------
gemfire-core/src/main/java/com/gemstone/gemfire/cache/query/internal/CompiledSelect.java (line 361) <https://reviews.apache.org/r/36477/#comment147238> javadoc update with new param - anilkumar gingade On July 14, 2015, 3:53 p.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36477/ > ----------------------------------------------------------- > > (Updated July 14, 2015, 3:53 p.m.) > > > Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, > Lynn Gallinat, and Qihong Chen. > > > Repository: geode > > > Description > ------- > > query object in query execution was being used without being set for nested > queries. This caused NullPointerExceptions. > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/cache/query/internal/CompiledSelect.java > be06404 > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/PartitionedRegion.java > bc164a8 > > gemfire-core/src/test/java/com/gemstone/gemfire/cache/query/partitioned/PRQueryDUnitTest.java > caeac58 > > gemfire-core/src/test/java/com/gemstone/gemfire/cache/query/partitioned/PRQueryJUnitTest.java > 97bf2df > > Diff: https://reviews.apache.org/r/36477/diff/ > > > Testing > ------- > > ./gradlew build (4 failing tests unrelated. Rerun and passed except for > AnalyzeSerializables but I think that's from previous check ins) > > > Thanks, > > Jason Huynh > >
