----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60935/#review180828 -----------------------------------------------------------
Fix it, then Ship it! fix then ship geode-core/src/main/java/org/apache/geode/pdx/internal/PeerTypeRegistration.java Line 532 (original), 531 (patched) <https://reviews.apache.org/r/60935/#comment256114> A minor nit: change "hasPersistentRegions" to "hasPersistentRegion" since we only check that is has at least one persistent region. geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionTestBase.java Line 391 (original), 391 (patched) <https://reviews.apache.org/r/60935/#comment256113> You should remove the change to this file before checking in just to reduce the number of changed files. - Darrel Schneider On July 17, 2017, 6:20 p.m., anilkumar gingade wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60935/ > ----------------------------------------------------------- > > (Updated July 17, 2017, 6:20 p.m.) > > > Review request for geode, Darrel Schneider, Eric Shu, Lynn Gallinat, and Nick > Reich. > > > Repository: geode > > > Description > ------- > > The pdx registry needs to be persisted to use pdx with persistent region. > Currently while creating pdx the system checks, to see if there is a disk > store with data instead of looking for persistent region in the system. In > cases where persistent region is created and dropped, the system doesn't > allow creating pdx witout setting pdx persistence. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java > 2dda38c > geode-core/src/main/java/org/apache/geode/internal/cache/InternalCache.java > 4c229db > > geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/CacheCreation.java > 61f91a0 > > geode-core/src/main/java/org/apache/geode/pdx/internal/PeerTypeRegistration.java > 065255b > > geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionTestBase.java > 45c6120 > geode-core/src/test/java/org/apache/geode/pdx/PdxAttributesJUnitTest.java > ef15cd5 > > > Diff: https://reviews.apache.org/r/60935/diff/1/ > > > Testing > ------- > > Added new test. > Precheckin started. > > > Thanks, > > anilkumar gingade > >