Skip to content
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 #15062: store run agent batch may never catch up on very loaded system because of inefficient index and the way it handles reports #2256

Conversation

ncharles
Copy link
Member

… system because of inefficient index and the way it handles reports
@ncharles ncharles force-pushed the bug_15062/store_run_agent_batch_may_never_catch_up_on_very_loaded_system_because_of_inefficient_index_and_the_way_it_handles_reports branch from b878026 to fba640a Compare June 12, 2019 11:13
@ncharles
Copy link
Member Author

Commit modified

@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder/pull/2256
-- Your faithful QA
Kant merge: "In law a man is guilty when he violates the rights of others. In ethics he is guilty if he only thinks of doing so."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/11196/console)

@fanf
Copy link
Member

fanf commented Jun 12, 2019

OK, merging this PR

@fanf fanf merged commit fba640a into Normation:branches/rudder/5.0 Jun 12, 2019

// Get max ID before a datetime
def getMaxIdBeforeDateTime(fromId: Long, before: DateTime): Box[Long] = {
(query[Long](s"select max(id) as id from RudderSysEvents where where id > ${fromId} and executionTimeStamp < '${new Timestamp(before.getMillis)}''").
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra ' here

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra where also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants