-
Notifications
You must be signed in to change notification settings - Fork 73
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.
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
Fixes #16083: Big memory usage when fetching/writing ndoe configuration and expected reports #2569
Conversation
PR updated with a new commit |
f7e2394
to
4165f67
Compare
Commit modified |
...core/src/main/scala/com/normation/rudder/repository/jdbc/ExpectedReportsJdbcRepository.scala
Outdated
Show resolved
Hide resolved
4165f67
to
4c6e32f
Compare
Commit modified |
PR updated with a new commit |
6ee2b62
to
c4d6c85
Compare
Commit modified |
c4d6c85
to
af26b8b
Compare
Commit modified |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about that one. It means that we loose the atomicity property and so, we could be in a case where we have a failed generation with some commited (and not other) expected reports.
AFAIU, the problem is the query (ie string) size. What could be a better solution, if available, is to strart transaction, stream changes, commit transaction - but I don't know how to do it. I propose to postpone that one for now to study that possibility.
I can split the PR in two, one for the read, that don't need to be atomic, and post-pone the writing part that need atomicity |
PR updated with a new commit |
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
…on and expected reports
f07a8f5
to
bc70f08
Compare
https://issues.rudder.io/issues/16083