Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Update force flush default #261

Merged
merged 4 commits into from
Sep 7, 2021

Conversation

Mrod1598
Copy link
Contributor

@Mrod1598 Mrod1598 commented Sep 7, 2021

Resolves #257

@Mrod1598 Mrod1598 requested a review from a team September 7, 2021 15:47
@codecov
Copy link

codecov bot commented Sep 7, 2021

Codecov Report

Merging #261 (2ad5b79) into main (cfff3b0) will not change coverage.
The diff coverage is 100.0%.

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #261   +/-   ##
=====================================
  Coverage   75.7%   75.7%           
=====================================
  Files         95      95           
  Lines       4453    4453           
=====================================
  Hits        3371    3371           
  Misses       752     752           
  Partials     330     330           
Impacted Files Coverage Δ
operator/helper/multiline.go 89.5% <100.0%> (ø)

@djaglowski djaglowski merged commit d0c9d8d into open-telemetry:main Sep 7, 2021
@djaglowski djaglowski deleted the update-force-flush-default branch September 7, 2021 19:38
dmitryax added a commit to dmitryax/opentelemetry-collector-contrib that referenced this pull request Oct 20, 2021
Update default values in `filelog`, `tcplog` and `udplog` receivers according to the recent changes in the log-collection library:
- open-telemetry/opentelemetry-log-collection#147
- open-telemetry/opentelemetry-log-collection#261
bogdandrutu pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this pull request Oct 20, 2021
Update default values in `filelog`, `tcplog` and `udplog` receivers according to the recent changes in the log-collection library:
- open-telemetry/opentelemetry-log-collection#147
- open-telemetry/opentelemetry-log-collection#261
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

force_flush_period should default to non-zero value
2 participants