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.

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

Jump to bottom

Configurable sleep time for Agrona agent #77

Closed
tkowalcz opened this issue Oct 20, 2021 · 0 comments · Fixed by #79
Closed

Configurable sleep time for Agrona agent #77

tkowalcz opened this issue Oct 20, 2021 · 0 comments · Fixed by #79
Assignees
@tkowalcz
Labels
core Relates to Core component

Comments

@tkowalcz
Copy link
Owner

ReadingLogBufferAndSendingHttp thread sleeps only for 1ms between runs. This can consume significant part of CPU time. Make the delay configurable.
@tkowalcz tkowalcz added the core Relates to Core component label Oct 20, 2021
@tkowalcz tkowalcz self-assigned this Oct 20, 2021
tkowalcz added a commit that referenced this issue Oct 21, 2021
@tkowalcz
Re #77: Renamed LogBufferAgent and its thread (ReadingLogBufferAndSen…
7883f48 
…dingHttp) to LogShipper. Added option to configure time it sleeps between reading from log buffer to not hog CPU. Also now default is 25ms instead of hardcoded 1ms previously.
tkowalcz added a commit that referenced this issue Oct 21, 2021
@tkowalcz
Re #77: Renamed LogBufferAgent and its thread (ReadingLogBufferAndSen…
7ae613a 
…dingHttp) to LogShipper. Added option to configure time it sleeps between reading from log buffer to not hog CPU. Also now default is 25ms instead of hardcoded 1ms previously.
tkowalcz added a commit that referenced this issue Oct 21, 2021
@tkowalcz
Re #77: Improved property naming (logShipperIntervalMillis -> logShip…
bca115e 
…perWakeupIntervalMillis) and implemented missing Logback part.
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Changed default logShipperWakeupIntervalMillis to 10ms. Added…
6cd29a1 
… documentation.
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Explicitly using Loki 2.3.0 docker image.
d732c1f
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging test failure on CircleCI.
8b8befc
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging test failure on CircleCI.
26273f5
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging and making tests run faster by disabling some ot them.
c6f1e7b
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging
79148ac
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging
c008372
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging
42358a1
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Debugging
8ee05e3
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Cleanup after debugging and bugfix (incorrect order of argume…
52e2381 
…nts).
tkowalcz added a commit that referenced this issue Oct 23, 2021
@tkowalcz
Re #77: Cleanup after debugging
660b958
@tkowalcz tkowalcz linked a pull request Oct 23, 2021 that will close this issue
77 agrona sleeping more #79
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tkowalcz tkowalcz

Labels
core Relates to Core component
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

77 agrona sleeping more tkowalcz/tjahzi
1 participant
@tkowalcz