[QFJ-522] Engine incorrectly assumes CstmApplVerID implies Session and DataDictionary uniqueness Created: 10/May/10 Updated: 15/Nov/12 Resolved: 24/Apr/11 |
|
Status: | Closed |
Project: | QuickFIX/J |
Component/s: | Engine |
Affects Version/s: | 1.4.0, Future Releases |
Fix Version/s: | None |
Type: | Bug | Priority: | Default |
Reporter: | Brent Bradbury | Assignee: | Grant Birchmeier |
Resolution: | Fixed | Votes: | 1 |
Labels: | None | ||
Environment: |
Centos 5.3, Sun JDK 1.6 |
Attachments: | CORRECTED_executorExampleAddendum.diff cstmApplVerIDFix.diff executorExampleAddendum.diff |
Description |
To show bug: Why it happens: The fix: |
Comments |
Comment by Brent Bradbury [ 10/May/10 ] |
Apply the patch to trunk/core/src. I'm not sure how the unit tests shake out, because I don't see where the specifics of failures/errors is shown? |
Comment by Brent Bradbury [ 10/May/10 ] |
Also, the patch is against rev 949 of trunk SVN. |
Comment by Grant Birchmeier [ 11/Aug/10 ] |
If you patch cstmApplVerIDFix.diff on a clean checkout and build, or if you patch then "ant clean" then rebuild, you'll get a compile error for the "executor" example. It needed to be updated to match an updated interface, which is a one-line fix described in attached patch executorExampleAddendum.diff. |
Comment by Grant Birchmeier [ 11/Aug/10 ] |
Oops. I messed up my patch file. Use this one. Apologies. |
Comment by Grant Birchmeier [ 24/Apr/11 ] |
code fix in trunk r1020 |