splunk / pipelines

Concurrent processing pipelines in Go.
MIT License
21 stars 8 forks source link

test: ErrorSink; fix: small error #34

Closed kalexmills-splunk closed 2 years ago

kalexmills-splunk commented 2 years ago

Unit tests ErrorSink and fixes a data race. Users of ErrorSink should now retrieve all errors which had started to be reported during any call to errs.All().

Calling errs.All() in the middle of a highly-contentious pipeline could now block indefinitely.

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2900019232


Totals Coverage Status
Change from base Build 2899803702: 3.3%
Covered Lines: 335
Relevant Lines: 351

💛 - Coveralls