axiomhq / cli

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

Sleep when no input is available while streaming #98

Closed lukasmalkmus closed 2 months ago

lukasmalkmus commented 3 years ago

Sleep on some loops when no inputs is available.

Closes #90.

codecov[bot] commented 3 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@53dd7df). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage        ?   14.22%           
=======================================
  Files           ?        9           
  Lines           ?      506           
  Branches        ?        0           
=======================================
  Hits            ?       72           
  Misses          ?      418           
  Partials        ?       16           

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 53dd7df...aa0fb44. Read the comment docs.

ImLunaHey commented 1 year ago

@lukasmalkmus is this PR still valid?

lukasmalkmus commented 1 year ago

Technically, yes. Apparently this got lost as no one else ever reported any issues. My idea was to test this branch against main. I couldn't imagine Go really uses up a lot of CPU when waiting on a select statement. By "test" I mean to run two long running jobs and monitor CPU usage.