Upgrading Mina to 1.1.7 would allow the removal of backport-util-concurrent.jar from the dependencies.
This patch replaces Mina 1.0.1 with Mina 1.1.7. The Queue and BlockingQueue classes from Mina were replaced by their equivalent in java.util.concurrent. VmPipeIdleStatusChecker has been removed since the issue it worked around has been fixed in Mina 1.1.7.
Mina 1.1.7 can be downloaded here :
There are some unused classes in the mina package that could also be removed:
Additional patch to account for the latest changes in AddressBasedDestinationTest by rajith (rev 934563)
Additional patch to update the POMs
Thanks for the patches I'll take a look at these over the next few days.
Martin, let me know if you need an updated patch when you are ready to review this change, I'll provide one.
Updated patch for the latest trunk
Is there any chance to get this into the 0.8 release? If someone can review this issue I'll happily update my patch.
I updated the patch, it's now available as a branch on Github. Please let me know if this format is not suitable.
Moving to 0.9, due to insufficient testing
I refreshed the patch on Github:
I hope to incorporate this upgrade into some upcoming work I will be doing. Can I ask if you are already using the patched source with this change in place?
Thank you Robbie. I don't use the patch, I use a trimmed down client that doesn't rely on Mina.
Emmanuel's patch had gone stale over the last 15 months (removal of inVM, pom dep changes etc). I've created a new patch which achieves the same result against the current codebase.
Thanks Emmanuel for the original patch (sorry we didnt get this done earlier), and Keith for the newer one.
Keith's updated patch is now applied. I noticed just after committing it that the pom dependency stubs still needed to be renamed, so I have also done that.
Closing this JIRA out now.