Skip to content

backend/http: add "intelligent" graph locking behavior#84

Open
RuthgerD wants to merge 1 commit intomasterfrom
burstprotec
Open

backend/http: add "intelligent" graph locking behavior#84
RuthgerD wants to merge 1 commit intomasterfrom
burstprotec

Conversation

@RuthgerD
Copy link
Copy Markdown
Collaborator

keeps track of the average delta between sleeps, if the delta is higher we sleep again until it reaches the average again.

this helps in situations where large amount of events are coming in for a short period, constant load effectively behaves the same as before.

we can extend this to adjusting the sleep time as well to help out constant high load situations.

@RuthgerD RuthgerD added Performance Performance related matters Backend labels Jan 10, 2022
@RuthgerD RuthgerD self-assigned this Jan 10, 2022
@RuthgerD
Copy link
Copy Markdown
Collaborator Author

also fixes a "bug" where we did not check if the key exists in ChunkedGraph, this is probably a performance regression :/

Copy link
Copy Markdown
Collaborator

@SyncInProgress SyncInProgress left a comment

Choose a reason for hiding this comment

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

LGTM

@RuthgerD
Copy link
Copy Markdown
Collaborator Author

I feel this requires a bit more testing before we can merge this as it can potentially mess up the entire backend :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Performance Performance related matters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants