[QFJ-321] PersistMessages config parameter default value is wrong in the QuickFIX/J User Manual Created: 01/Jul/08 Updated: 15/Nov/12 Resolved: 01/Jul/08 |
|
| Status: | Closed |
| Project: | QuickFIX/J |
| Component/s: | Documentation, Engine |
| Affects Version/s: | 1.3.1 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Default |
| Reporter: | Baxter Solutions | Assignee: | Unassigned |
| Resolution: | Not a bug | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Description |
|
The QuickFIX/J User Manual says that the default value of the PersistMessages parameter is "N" (false). But in the DefaultSessionFactory its default value is true (line 150): boolean persistMessages = getSetting(settings, sessionID, Session.SETTING_PERSIST_MESSAGES, true); If this parameter is set to true, then there is a memory leak in the FileStore class. The size of HashMap messageIndex is growing all the time. The reset() method does not clear this map. |
| Comments |
| Comment by Steve Bate [ 01/Jul/08 ] |
|
Are you using the documentation that is shipped with QFJ or the online documentation? The online documentation is sometimes a bit out of date. If it's a problem with the online documentation I'll be sure to update it when I release 1.3.2 (any day now). The PersistMessages docs should be correct in the 1.3.1 distribution. The value was changed in SVN revision 748 and the 1.3.1 release tag is revision 786. The messageIndex is cleared by reset. reset() -> initialize() --> initializeCache() --> initializeMessageIndex() --> messageIndex.clear() ... |