----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/ -----------------------------------------------------------
Review request for geode, Eric Shu and Lynn Gallinat. Bugs: GEODE-3027 https://issues.apache.org/jira/browse/GEODE-3027 Repository: geode Description ------- The new resolver uses a fixed delimiter of "|". In this version of the resolver you can not configure the delimiter. "|" was chosen over ":" because it seemed less likely that the desired prefix string would contain it. Since the delimiter can not be configured it is important that it would not be a common one to occur in a key string. Diffs ----- geode-core/src/main/java/org/apache/geode/cache/partition/StringPrefixPartitionResolver.java PRE-CREATION geode-core/src/test/java/org/apache/geode/cache/partition/StringPrefixPartitionResolverJUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/59849/diff/1/ Testing ------- precheckin Thanks, Darrel Schneider