Add new config::AlterSystemParameters struct which is responsible
for definitions of system parameters, their default values and mapping
to sbroad type.

This is a big step towards moving all system parameters from
_pico_property into another system table.

We pass a timeout into the connection pool call which should be enough

Maksim Kaitmazian authored 5 months ago and Maksim Kaitmazian committed 5 months ago

This commit resolves the performance issues observed in pgbench
benchmark*. Because of the Nagle's algorithm, every message was read with
50ms delay, while the query execution took only 1ms. By disabling the
algorithm via TCP_NODELAY option, we achieve 50 times performance
improvement for a single client in simple query mode.

* https://git.picodata.io/picodata/picodata/picodata/-/issues/1002

Denis Smirnov authored 5 months ago and Dmitry Rodionov committed 5 months ago

There was a bug when SQL statistics LRU removed some old query
from the _sql_query table. The query ref_counter field in unsigned,
but the code decremented it below zero in the space on eviction.
That produced an error (fixed).

Should read footer as:

References #991
Closes #943

Дмитрий Кольцов authored 5 months ago and Dmitry Rodionov committed 5 months ago

Due to the bug in RPC (see https://git.picodata.io/picodata/picodata/picodata/-/issues/991)
RPC call timeout does not account TCP connection establishment.
So a request to a peer without a connection was not aborted
due to timeout.
Force a timeout directly for a future returned by RPC to avoid this

Closes #991

Дмитрий Кольцов authored 5 months ago and Dmitry Rodionov committed 5 months ago

Previously webUI failed to retrieve data about
instances if one of those was dead. The cause was
that webUI HTTP request timeout was less than
get_instances_data timeout.

References #943

Arseniy Volynets authored 5 months ago and Denis Smirnov committed 5 months ago

- update sbroad submodule to commit with fix

Дмитрий Кольцов authored 5 months ago and Georgy Moshkin committed 5 months ago

All migrations are explicitly listed in manifest.yaml,
so there is no much sense in strict extension check

Closes #990