Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1114945

Summary: Hotrod client TransportFactory is not backwards compatible
Product: [JBoss] JBoss Data Grid 6 Reporter: Jakub Markos <jmarkos>
Component: ServerAssignee: Tristan Tarrant <ttarrant>
Status: CLOSED NOTABUG QA Contact: Martin Gencur <mgencur>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.3.0CC: galder.zamarreno, jdg-bugs, mhusnain, mmarkus
Target Milestone: ---   
Target Release: 6.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
HotRod client's TransportFactory is not backward compatible even though the API was initially documented as public. The TransportFactory interface is really an internal API and users should not try to extend it in the future.
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-01 10:21:58 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:

Comment 2 Galder ZamarreƱo 2014-07-01 10:21:58 UTC
This is really inner API, I really don't know why it was exposed as configurable in the first place. The changes are necessary to implement a lot of the Hot Rod client improvements coming up in Infinispan 7.0.

Comment 3 Mircea Markus 2014-07-01 10:37:34 UTC
Indeed exposing the transport factory is something quite internal and shouldn't have been done in the first place. I don't expect people to extend it - in fact that interface has been changed in 6.2 as well, without any concerns - but for now would be a bit risky to remove it entirely. Let's keep it there just in case - trying to keep backward compatibility in this case is a significant effort and not worth making.

Comment 4 Martin Gencur 2014-07-04 08:14:02 UTC
Probably needs to be documented in release notes. Adding the flag.

Comment 5 Misha H. Ali 2014-09-09 03:49:32 UTC
This was documented in the 6.3.0 Release Notes. Since this was closed NOTABUG, I'll assume we don't want this documented for any other release's Release Notes, hence removing the flag.