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

File input should not complain about files already being closed #174

Merged
merged 1 commit into from
Jun 9, 2021

Conversation

djaglowski
Copy link
Member

@djaglowski djaglowski commented Jun 9, 2021

File input generates errors when closing files, because they are being closed twice.

@codecov
Copy link

codecov bot commented Jun 9, 2021

Codecov Report

Merging #174 (db69ec2) into main (9050cb7) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

❗ Current head db69ec2 differs from pull request most recent head dbc74b6. Consider uploading reports for the commit dbc74b6 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##            main    #174     +/-   ##
=======================================
- Coverage   75.6%   75.6%   -0.1%     
=======================================
  Files         95      95             
  Lines       4362    4363      +1     
=======================================
- Hits        3301    3300      -1     
- Misses       738     739      +1     
- Partials     323     324      +1     
Impacted Files Coverage Δ
operator/builtin/input/file/file.go 70.6% <0.0%> (-1.2%) ⬇️

@djaglowski
Copy link
Member Author

#175 Fixes codecov config. Will rebase when it is merged.

@djaglowski djaglowski force-pushed the quiet-close-error branch from db69ec2 to dbc74b6 Compare June 9, 2021 21:01
@djaglowski djaglowski marked this pull request as ready for review June 9, 2021 21:24
@djaglowski djaglowski requested a review from a team June 9, 2021 21:24
@djaglowski djaglowski merged commit d988d23 into open-telemetry:main Jun 9, 2021
@djaglowski djaglowski deleted the quiet-close-error branch June 9, 2021 21:35
@djaglowski djaglowski mentioned this pull request Jun 9, 2021
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.

2 participants