-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59849/
-----------------------------------------------------------
(Updated June 6, 2017, 11:54 a.m.)
Review request for geode, Eric Shu and Lynn Gallinat.
Changes
-------
removed delimiter from getName
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 (updated)
-----
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/2/
Changes: https://reviews.apache.org/r/59849/diff/1-2/
Testing
-------
precheckin
Thanks,
Darrel Schneider
