Skip to content

fix oom. if config resendRequestChunkSize. Loading data is batch #777

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 8 commits into from
Closed

fix oom. if config resendRequestChunkSize. Loading data is batch #777

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
17a1774
fix oom. if config resendRequestChunkSize. Loading data is batch
Mar 18, 2024
628baa0
fix oom. if config resendRequestChunkSize. Loading data is batch
Mar 18, 2024
e0c1d9e
fix oom. if config resendRequestChunkSize. Loading data is batch …
Mar 18, 2024
e47e651
add case test
Mar 19, 2024
5a18e39
add case test
Mar 19, 2024
b96e9b3
add case test
Mar 31, 2024
074f73c
rollback case test
May 25, 2024
fc2c7fc
Merge branch 'master' into fix-oom
Feb 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion quickfixj-core/src/main/java/quickfix/Session.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2345,7 +2345,11 @@ private void nextLogon(Message logon) throws FieldNotFound, RejectLogon, Incorre

private void resendMessages(Message receivedMessage, int beginSeqNo, int endSeqNo)
throws IOException, InvalidMessage, FieldNotFound {

int lastEndSeqNoSent = resendRequestChunkSize == 0 ? endSeqNo : beginSeqNo + resendRequestChunkSize - 1;
if (lastEndSeqNoSent > endSeqNo) {
lastEndSeqNoSent = endSeqNo;
}
endSeqNo = lastEndSeqNoSent;
final ArrayList<String> messages = new ArrayList<>();
try {
state.get(beginSeqNo, endSeqNo, messages);
Expand Down