[QFJ-664] Upgrade MINA from 1.1.8 to 2.0.x Created: 25/Jan/12 Updated: 02/Apr/15 Resolved: 03/Jan/14 |
|
Status: | Closed |
Project: | QuickFIX/J |
Component/s: | Networking |
Affects Version/s: | 1.5.2, 1.5.3 |
Fix Version/s: | 1.6.0 |
Type: | Improvement | Priority: | Default |
Reporter: | oliver jehle | Assignee: | Christoph John |
Resolution: | Fixed | Votes: | 6 |
Labels: | None |
Attachments: | actual_patch_v02.txt quickfix-1.5.3-mina2.path | ||||||||||||||||||||
Issue Links: |
|
Description |
Upgrade quickfix/j to the mina 2.x release NB: In the meantime MINA version is now 2.0.9 (https://github.com/quickfix-j/quickfixj/commit/93f263a2096d9f3a102cfb04b78a76515810927d) |
Comments |
Comment by Attila-Mihaly Balazs [ 27/Feb/12 ] |
The patch from |
Comment by Jörg Thönnes [ 28/Feb/12 ] |
In reply to comment #1: Sounds good. Please provide feedback as soon as you have some. |
Comment by oliver jehle [ 05/Mar/12 ] |
sorry, late delay... i will rediff on the latest tree and post the patch with all cleanups. the problem is, that i have to split out the proxy hack stuff und the mina stuff. i hope i can work tomorrow in train on the diff, so i can repost the patch tomorrow or on wednesday. |
Comment by Jörg Thönnes [ 06/Mar/12 ] |
In reply to comment #3: That's fine, Oliver. I am also short of time due to my work. |
Comment by oliver jehle [ 13/Mar/12 ] |
latest patch and last one.... please test this patch in your UAT environment. there is only a small diff to the v01 patch. only some System.out and System.err and Log.info are removed. rest ist stable and the ant test was successfull. don't forget to replace the mina-1.1.7 with the mina-2.0.4 or higher libs... |
Comment by Christoph John [ 01/Aug/12 ] |
Unfortunately it seems that we will have to wait until at least 1.6.0 to incorporate this change since we need to discontinue the JDK1.4 support of QF/J when we want to use MINA2. MINA2 is only available for JDK5+ and the QFJ code needs to be changed to work with MINA2. |
Comment by Attila-Mihaly Balazs [ 19/Mar/13 ] |
This is an updated version of the previous patch, done against quickfix/j 1.5.3. I would like to mention that at this point Java 1.4 is ancient history and not even Java 1.6 is publicly supported anymore. Along this line of reasoning you should run java 1.7 because (other than common sense - which is not all that common):
|
Comment by Christoph John [ 20/Mar/13 ] |
Thanks for the patch. I agree that we should definitely use a recent JDK. I will send a message to the quickfixj-users group. |
Comment by Attila-Mihaly Balazs [ 17/Jun/13 ] |
One more reason to remove 1.5 and lower: the JDK itself will remove support for them: http://openjdk.java.net/jeps/182 |
Comment by Aaron Li [ 11/Jul/13 ] |
Applied patch quickfix-1.5.3-mina2.path and built without issues. Can someone point me how to make a change to quickfix/j and let it pick up proxy settings and establish connection via socks v4 proxy server? Tried returning a ProxyConnector with ProxyIoSession set from ProtocolFactory.createIoConnector() but got no luck. Can't even pass through jSocks tests. Very frustrated. Any help is highly appreciated. Aaron |
Comment by oliver jehle [ 11/Jul/13 ] |
i did only tests with http proxies. socks i dont know if mina support it, perhaps passing socks option but as you can see, the patch is old and never |
Comment by Niall [ 18/Jul/13 ] |
We have a dependency on both quickfixj & apache qpid - but qpid uses mina 2.0 which clashes with quickfixj's dependency on mina 1.0. So would be really good if quickfixj could move to mina 2.0 |
Comment by Christoph John [ 03/Jan/14 ] |
Committed as http://sourceforge.net/p/quickfixj/code/1131/ and http://sourceforge.net/p/quickfixj/code/1132/. |
Comment by Christoph John [ 03/Jan/14 ] |
With the new MINA libraries some deadlock situations on disconnect are now more prominent than before. This will be handled in |