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

Make buffer max chunk delay reconfigurable on the fly #313

Merged
merged 3 commits into from
May 24, 2021

Conversation

djaglowski
Copy link
Member

No description provided.

@djaglowski
Copy link
Member Author

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.4137858 -0.15520668 133.4022 +4.638336
1 5000 5.137905 -0.63812304 134.50835 +1.5639801
1 10000 9.344911 -1.0173769 142.17012 -1.6039886
1 50000 46.466026 -5.7422295 176.21417 +0.531662
1 100000 88.00082 -9.844452 223.6685 -15.027344
10 100 1.913861 -0.3448105 131.40114 -1.0662689
10 500 5.3966355 -0.87931585 137.7694 -3.2637482
10 1000 11.396672 -0.89628124 149.38025 -1.6127472
10 5000 50.1228 -4.379532 180.06035 -4.0779877
10 10000 103.66129 -1.6695557 213.12904 -11.542297

@djaglowski
Copy link
Member Author

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.5344716 -0.034520864 127.31695 -1.4469223
1 5000 4.6035194 -1.1725087 135.46364 +2.5192719
1 10000 9.276029 -1.0862589 143.7923 +0.018188477
1 50000 44.72376 -7.484497 174.02815 -1.6543579
1 100000 89.31134 -8.533928 233.14305 -5.5527954
10 100 1.8621423 -0.3965292 136.7104 +4.242996
10 500 5.431067 -0.8448844 138.89763 -2.1355133
10 1000 12.552064 +0.25911045 144.488 -6.5049896
10 5000 49.36254 -5.1397896 180.06668 -4.0716553
10 10000 96.72426 -8.60659 217.10493 -7.5664062

@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #313 (c3bf914) into master (cd2cf8b) will decrease coverage by 0.36%.
The diff coverage is 28.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #313      +/-   ##
==========================================
- Coverage   69.66%   69.30%   -0.36%     
==========================================
  Files         119      119              
  Lines        6309     6333      +24     
==========================================
- Hits         4395     4389       -6     
- Misses       1446     1475      +29     
- Partials      468      469       +1     
Impacted Files Coverage Δ
operator/buffer/buffer.go 80.95% <ø> (ø)
operator/buffer/disk.go 65.27% <0.00%> (-3.13%) ⬇️
operator/buffer/memory.go 74.02% <57.14%> (-2.51%) ⬇️
operator/builtin/input/file/file.go 73.97% <0.00%> (-4.79%) ⬇️
operator/flusher/flusher.go 84.00% <0.00%> (-4.00%) ⬇️
operator/builtin/output/forward/forward.go 56.52% <0.00%> (-2.90%) ⬇️
operator/builtin/output/newrelic/newrelic.go 71.03% <0.00%> (-0.93%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cd2cf8b...c3bf914. Read the comment docs.

@djaglowski
Copy link
Member Author

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 1.4310278 -0.13796473 127.978714 -0.78515625
1 5000 4.931177 -0.844851 137.1646 +4.22023
1 10000 9.155215 -1.2070723 139.9507 -3.82341
1 50000 49.535633 -2.6726227 179.52869 +3.8461761
1 100000 95.226105 -2.6191635 229.29701 -9.398834
10 100 1.8965417 -0.3621298 131.62729 -0.8401184
10 500 5.8622537 -0.41369772 140.74258 -0.29055786
10 1000 11.517255 -0.77569866 145.42525 -5.567749
10 5000 54.971733 +0.4694023 181.10048 -3.037857
10 10000 96.916245 -8.414604 219.22226 -5.4490814

@djaglowski djaglowski marked this pull request as ready for review May 24, 2021 15:09
@djaglowski djaglowski merged commit eda97c7 into master May 24, 2021
@jsirianni jsirianni deleted the buffer-reconfig branch July 13, 2021 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants