Re: Review Request 52891: GEODE-538: Add check for persistent data recovery

2016-10-18 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52891/ --- (Updated Oct. 18, 2016, 8:25 p.m.) Review request for geode, anilkumar gingade,

Re: Review Request 52891: GEODE-538: Add check for persistent data recovery

2016-10-18 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52891/#review153129 --- Fix it, then Ship it! Ship It! geode-core/src/main/java/org/a

Re: Review Request 52891: GEODE-538: Add check for persistent data recovery

2016-10-18 Thread Ken Howe
> On Oct. 17, 2016, 7:49 p.m., anilkumar gingade wrote: > > Added the @throws PartitionOfflineException to the javadocs for the other (non-0-argument) execute methods. > On Oct. 17, 2016, 7:49 p.m., anilkumar gingade wrote: > > geode-core/src/main/java/org/apache/geode/cache/query/internal/De

Re: Review Request 52891: GEODE-538: Add check for persistent data recovery

2016-10-17 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52891/ --- (Updated Oct. 17, 2016, 11:15 p.m.) Review request for geode, anilkumar gingade

Re: Review Request 52891: GEODE-538: Add check for persistent data recovery

2016-10-17 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52891/#review152936 --- geode-core/src/main/java/org/apache/geode/cache/query/internal/De