Bug 1111159

Summary:	ConsistentHash and ConsistentHashFactory API contract unstable
Product:	[JBoss] JBoss Data Grid 6	Reporter:	Radim Vansa <rvansa>
Component:	Documentation	Assignee:	Christian Huffman <chuffman>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.3.0	CC:	chuffman, jdg-bugs, ttarrant, wburns
Target Milestone:	---	Keywords:	Reopened
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-01-30 19:46:36 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Radim Vansa 2014-06-19 11:40:38 UTC

I am not 100% sure whether these two classes should be public or not in JDG; in community these are public, and the interfaces can be implemented and used (we document that [1]). Still, they are not documented in API - it seems ISPN-4074 has not been correctly applied for jdg-6.3.x branch.

Then, [2] brings a new method into the ConsistentHash interface, breaking the API contract.

This should have effect only on customers who are using their own implementation of ConsistentHashFactory, though.


[1] http://docbuilder.usersys.redhat.com/12532/#Implementing_a_ConsistentHashFactory
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1099969

Comment 2 William Burns 2014-06-19 14:21:22 UTC

When I spoke with Tristan about this interface, we were thinking this interface was used so infrequently it wasn't a big deal.  This could be a false assumption, I don't know myself.

Comment 3 Radim Vansa 2014-06-20 14:18:36 UTC

I understand, Galder even removed the configuration option to set custom ConsistentHashFactory from 7.x-style XML configuration. I really don't know how "big deal" is it to break API in *any* way, therefore, I had to report that.

You always have the option to close the bugzilla as 'won't fix'.

Comment 5 Christian Huffman 2019-01-30 19:46:36 UTC

I'm closing this issue out due to age. If it persists, file a corresponding JIRA.