----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59849/#review177064 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/cache/partition/StringPrefixPartitionResolver.java Lines 60 (patched) <https://reviews.apache.org/r/59849/#comment250629> Does getName() needs the Delimiter being included? If it is used in the equals(), can we just check getDelimiter() in the equals method? - Eric Shu On June 6, 2017, 5:44 p.m., Darrel Schneider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59849/ > ----------------------------------------------------------- > > (Updated June 6, 2017, 5:44 p.m.) > > > 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 > >
