jdeppe-pivotal commented on a change in pull request #6117:
URL: https://github.com/apache/geode/pull/6117#discussion_r592786983
##########
File path:
geode-redis/src/main/java/org/apache/geode/redis/internal/data/ByteArrayWrapper.java
##########
@@ -107,6 +115,39 @@ public int compareTo(ByteArrayWrapper other) {
return arrayCmp(value, other.value);
}
+ /**
+ * Used by the {@link RedisPartitionResolver} to map slots to buckets.
Supports using hashtags
+ * in the same way that redis does.
+ *
+ * @see <a href="https://redis.io/topics/cluster-spec";>Redis Cluster Spec</a>
+ */
+ public synchronized Object getRoutingId() {
Review comment:
Currently the `ByteArrayWrapper` is used for both keys and values. I
have another PR that will introduce a `RedisKey` which will do exactly what
you're suggesting.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
