search

axiomhq / cli

The power of Axiom on the command line.
https://axiom.co
MIT License
41 stars 11 forks source link

Scan in a go func when --flush-every is set #25

Closed bahlo closed 3 years ago

bahlo commented 3 years ago

This fixes a bug where no data is sent if the stream is stalled. With this change we scan in another go func and provide individual lines on a channel, so if the stream blocks we're still able to go into the ticker branch.

Fixes #24

codecov[bot] commented 3 years ago

Codecov Report

Merging #25 (13c8ddc) into main (ede31c9) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   14.81%   14.81%           
=======================================
  Files           9        9           
  Lines         486      486           
=======================================
  Hits           72       72           
  Misses        399      399           
  Partials       15       15

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 ede31c9...2335301. Read the comment docs.