add cpu and memory overload warnings#86838
Conversation
|
Workflow [PR], commit [174dd50] Summary: ❌
|
|
To be honest, I'm not sure that this is the client responsibility to show this, since for instance I use persisten client sessions, while warings are shown only at start so if I am spawning client at the server start more or less, then I will not see this warnings But it looks cheap and there is already So I guess it is OK to have them in client as well. |
The idea would be that clients, but not necessarily clickhouse-client such a cloud UI, can check system.warnings and show them / alert if needed. |
06984bb to
c6fee19
Compare
88c52bd to
174dd50
Compare
|
Test failures not related:
|
Add overload warnings for cpu and memory to
system.warningstable.Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):
Add warnings for cpu and memory to
system.warningstable.Documentation entry for user-facing changes