1. make it non-nullable
2. fix tests

- update sbroad submodule to commit supporting
inferring not null constraint on primary key
columns
- remove redundant not null constraint from
most integration tests

Kurdakov Alexander authored 9 months ago and Alexander Kurdakov committed 8 months ago

following sections were renamed:
tiers -> tier
peers -> peer

Example:

https://git.picodata.io/picodata/picodata/picodata/-/jobs/189390

Actix is not it brings vendored C dependencies which certification
doesnt like. Because of the same reason swith away from rustls

Return value of a function 'space_cache_find' is dereferenced at
trigger.c:427 without checking for NULL, but it is usually checked for
this function (32/34).

Expression 'ub', used as divisor at lj_vmmath.c:69, may have a zero
value when variable 'b' has value '0'. Check at lj_vmmath.c:68 does not
prevent this error.

An element of array 'nc->stack' is accessed by an index with possible
negative value -2 at lj_opt_narrow.c:395. This may lead to a buffer
underflow. Index variable is assigned the value lj_opt_narrow.c:395.

There were another similar warning on the same line. Both should be
fixed.

Return value of a function 'lj_tab_getinth' is dereferenced at
lj_ccallback.c:540 without checking for NULL, but it is usually checked
for this function (14/15).

Possible integer overflow: right operand is tainted. An integer
overflow may occur due to arithmetic operation (multiplication)
between value '24' and variable 'sym_cnt', where the value comes
from 'sizeof(Elf64_Sym)' and when 'sym_cnt' is tainted
{ [1, 18446744073709551615] }

Warning:
Return value of function 'json_lexer_next_token', called at
key_def.c:222, is not checked, but it is usually checked for this
function (12/13).

Warning:
After having been compared to a NULL value at lemon.c:1880, pointer
'argv' is dereferenced at lemon.c:1881 by calling function 'strlen'.

Close: https://git.picodata.io/picodata/picodata/picodata/-/issues/638

Make use of single instance fixture for non gostech and gostech
instances. Avoid flakiness caused by unsynchronized shutdown of audit
log server and messages lookup.

Close: https://git.picodata.io/picodata/picodata/picodata/-/issues/644

The patch changes how ports are assigned to nodes started in tests.
Previously port range was given to any xdist worker and each test chose
ports from the beginning of the range which can lead to conflicts. So
with the patch tests can no longer have interleaving port ranges.

Additionally ports were previously checked but the check was failing the
whole test in case of a failure. With the patch occupied ports are
skipped instead.

The patch also fixes a source of flakiness in PluginReflection
PluginReflection

- rename it to proc_sql_dispatch
- modify args so that it's more suitable for public RPC API