Fix high CPU usage when polling data from a Logstash diagnostic path (#13)

edmocosta · web-flow · commit 0883d99001e5 · 2025-04-12T09:50:17.000-03:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,6 @@
+## 0.7.1
+- Fixed high CPU usage when polling data from a Logstash diagnostic path (`--diagnostic-path`).
+
 ## 0.7.0
 - Logstash APIs are now fetched concurrently, improving the overall performance, and reducing UI lags when pipelines are too complex or contains several components. [#12](https://github.com/edmocosta/tuistash/pull/12)
 
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -7,7 +7,7 @@ documentation = "https://github.com/edmocosta/tuistash"
 keywords = ["logstash", "tui", "cli", "terminal"]
 categories = ["command-line-utilities"]
 authors = ["Edmo Vamerlatti Costa <edinhow@gmail.com>"]
-version = "0.7.0"
+version = "0.7.1"
 edition = "2021"
 
 [dependencies]
diff --git a/src/cli/commands/tui/backend.rs b/src/cli/commands/tui/backend.rs
@@ -96,10 +96,11 @@ fn run_app<B: Backend>(terminal: &mut Terminal<B>, mut app: App) -> io::Result<(
 
         if app.sampling_interval.is_some() && last_tick.elapsed() >= tick_interval {
             app.on_tick();
-            last_tick = Instant::now();
         }
         if app.should_quit {
             return Ok(());
         }
+
+        last_tick = Instant::now();
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -96,10 +96,11 @@ fn run_app<B: Backend>(terminal: &mut Terminal<B>, mut app: App) -> io::Result<(`
`96`	`96`
`97`	`97`	`if app.sampling_interval.is_some() && last_tick.elapsed() >= tick_interval {`
`98`	`98`	`app.on_tick();`
`99`		`- last_tick = Instant::now();`
`100`	`99`	`}`
`101`	`100`	`if app.should_quit {`
`102`	`101`	`return Ok(());`
`103`	`102`	`}`
	`103`	`+`
	`104`	`+ last_tick = Instant::now();`
`104`	`105`	`}`
`105`	`106`	`}`