| [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 |