Skip to content

status: add parallel status execution - #1352

Merged
bigbes merged 2 commits into
masterfrom
g.rybakov/tntp-6354-status-opt
Aug 28, 2026
Merged

status: add parallel status execution#1352
bigbes merged 2 commits into
masterfrom
g.rybakov/tntp-6354-status-opt

Conversation

@sssciel

@sssciel sssciel commented Aug 18, 2026

Copy link
Copy Markdown
Member

Previously, the status call was called linearly on all instances,
which could lead to long delays. Now the calls are parallel.

On 200 instances, the total time gain is 12 times on
the test machine, from 5 seconds to 0.3 seconds.

Part of TNTP-6354

@sssciel
sssciel force-pushed the g.rybakov/tntp-6354-status-opt branch 6 times, most recently from d62f323 to 9976df5 Compare August 19, 2026 07:39
@sssciel
sssciel force-pushed the g.rybakov/tntp-6354-status-opt branch 5 times, most recently from 3da756f to d8c1038 Compare August 21, 2026 10:07
Comment thread cli/connector/connector.go Outdated
Comment thread cli/status/status.go

@patapenka-alexey patapenka-alexey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments above.

@sssciel
sssciel force-pushed the g.rybakov/tntp-6354-status-opt branch from d8c1038 to 5995481 Compare August 24, 2026 10:21
@sssciel
sssciel requested a review from Mockird31 August 24, 2026 10:21
@sssciel
sssciel force-pushed the g.rybakov/tntp-6354-status-opt branch from 5995481 to 6ebefd5 Compare August 27, 2026 14:57
Previously, the status call was called linearly on all instances,
which could lead to long delays. Now the calls are parallel.

Also status eval requests could stuck, so separate flag
--instance-timeout was added.

Part of TNTP-6354
Previously, the cluster config was re-read from disk for each instance,
which could lead to huge delays on hundreds of instances.
Now the cluster config is read once.

Closes TNTP-6354
@sssciel
sssciel force-pushed the g.rybakov/tntp-6354-status-opt branch from 6ebefd5 to ffe112b Compare August 27, 2026 14:58
@bigbes
bigbes merged commit a195447 into master Aug 28, 2026
34 of 35 checks passed
@bigbes
bigbes deleted the g.rybakov/tntp-6354-status-opt branch August 28, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants