On 26 Oct 2010, at 19:44, Mircea Markus wrote:

Hi,

ISPN-180 requires changes in ConsistentHash interface. This is a public interface in 4.x as its implementation is pluggable.

1. Is this something we want to do? I think we should doi it as a) the migration to new impl is relatively simple b) I don't think this is has a highly used feature 

2. The interface I need for implementing ISPN-180 looks something like: http://www.pastie.org/1250789

Last three methods were added, and two methods were removed: getDistance and isAdjacent 

How does this look to you?

One more thing - any reason why isn't part of the CH state? Seems like it is passed to the CH on almost all methods with the same value. Having it as state (together with List<Address>) would allow implementations to cache stuff.