geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Darrel Schneider <dschnei...@pivotal.io>
Subject Review Request 59849: add StringPrefixPartitionResolver
Date Tue, 06 Jun 2017 17:44:57 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message