Reduce the default log queue size for client commands #2176

tobim · 2022-03-31T08:47:03Z

The previous default of 1'000'000 lead to about 400MB of additional memory consumption. This is acceptable for the server but not for the often short-running client commands.

This PR also exposes the --log-queue-size option on the command line.

📝 Checklist

All user-facing changes have changelog entries.
The changes are reflected on docs.tenzir.com/vast, if necessary.

The previous default of 1'000'000 lead to about 400MB of additional memory consumption. This is acceptable for the server but not for the often short-running client commands.

lava

Makes sense to me.

tobim added 2 commits March 31, 2022 10:01

Expose the log queue size as a user option

967d23e

Reduce the default log queue size for client commands

3e55d68

The previous default of 1'000'000 lead to about 400MB of additional memory consumption. This is acceptable for the server but not for the often short-running client commands.

tobim added the enhancement label Mar 31, 2022

tobim requested a review from a team March 31, 2022 08:47

lava approved these changes Mar 31, 2022

View reviewed changes

Add a changelog entry for the logs tuning.

888a05e

tobim enabled auto-merge March 31, 2022 09:04

tobim merged commit 751da32 into master Mar 31, 2022

tobim deleted the story/sc-32547/tune-log-queue-size branch March 31, 2022 12:58

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Reduce the default log queue size for client commands #2176

Reduce the default log queue size for client commands #2176

tobim commented Mar 31, 2022 •

edited

Loading

lava left a comment

Reduce the default log queue size for client commands #2176

Reduce the default log queue size for client commands #2176

Conversation

tobim commented Mar 31, 2022 • edited Loading

📝 Checklist

lava left a comment

Choose a reason for hiding this comment

tobim commented Mar 31, 2022 •

edited

Loading