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

use google output 0.1.3 #265

Merged
merged 2 commits into from
Apr 2, 2021
Merged

use google output 0.1.3 #265

merged 2 commits into from
Apr 2, 2021

Conversation

jsirianni
Copy link
Member

Description of Changes

Use latest Google output https://github.com/observIQ/stanza/releases/tag/operator%2Fbuiltin%2Foutput%2Fgooglecloud%2Fv0.1.3

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Add a changelog entry (for non-trivial bug fixes / features)
  • CI passes

@jsirianni jsirianni requested a review from djaglowski April 2, 2021 14:32
djaglowski
djaglowski previously approved these changes Apr 2, 2021
@codecov
Copy link

codecov bot commented Apr 2, 2021

Codecov Report

Merging #265 (2e0b5f3) into master (4d0a4bc) will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #265      +/-   ##
==========================================
+ Coverage   71.63%   71.68%   +0.05%     
==========================================
  Files         103      103              
  Lines        5689     5689              
==========================================
+ Hits         4075     4078       +3     
+ Misses       1183     1177       -6     
- Partials      431      434       +3     
Impacted Files Coverage Δ
operator/builtin/output/otlp/otlp.go 65.43% <0.00%> (+3.70%) ⬆️

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 4d0a4bc...2e0b5f3. Read the comment docs.

@djaglowski
Copy link
Member

Log Files Logs / Second CPU Avg (%) CPU Avg Δ (%) Memory Avg (MB) Memory Avg Δ (MB)
1 1000 2.0689292 +0.34474957 123.40544 -3.9217377
1 5000 6.3104777 -0.5000763 132.94989 +1.8872528
1 10000 12.672759 -0.31033134 138.04688 -1.6330872
1 50000 58.346405 -3.3778877 168.06331 -10.294861
1 100000 116.748825 -2.1800919 231.94882 +0.14683533
10 100 2.3794239 -0.034541845 126.84308 -1.5098267
10 500 7.051936 +0.361948 134.68912 -0.21861267
10 1000 13.448597 -0.39698505 142.85413 +0.1376648
10 5000 62.74087 -1.4648323 182.10223 -0.7435303
10 10000 123.08933 -3.505272 227.16016 +4.736801

@jsirianni jsirianni merged commit 9202aba into master Apr 2, 2021
@jsirianni jsirianni deleted the google_output_0.1.3 branch April 2, 2021 14:46
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