Fix: Use channel semantics to manage `LogStream` lifecycle in `Tailer`.

google / mtail

extract internal monitoring data from application logs for collection in a timeseries database

Apache License 2.0

3.82k stars 378 forks source link

Fix: Use channel semantics to manage `LogStream` lifecycle in `Tailer`. #893

Closed jaqx0r closed 2 months ago

jaqx0r commented 2 months ago

Removes most of the LogStream interface functions, moving some functionality into the LogStream and using only closure of the lines channel to signal stream shutdown. All external notices are now handled by Context cancellation.

The stale_log_gc_interval flag is now deprecated as these are cleaned up at stream shutdown.

Issue: #199

github-actions[bot] commented 2 months ago

Unit Test Results

1 files 27 suites 8m 40s :stopwatch: 648 tests 647 :white_check_mark: 1 :zzz: 0 :x: 1 917 runs 1 914 :white_check_mark: 3 :zzz: 0 :x:

Results for commit f39cf7b6.