Previously picodata failed to drop spaces with grants on it ("the
space has grants"). Another problem was that we didn't clean
_pico_privileges grants when the space was dropped. Both problems
were fixed in the current commit.

The most obvious benefit is link checking. This is especially useful with rust-analyzer, because it allows to jump to definition of the item referenced in the doc comment.

The patch also makes CI to use `make lint` to avoid duplication.

The patch adds `password_min_length` parameter to `_pico_properties` space.
The parameter is taken into account in `pico.create_user` and 
`pico.change_password` lua functions.

Denis Smirnov authored 1 year ago and Yaroslav Dynnikov committed 1 year ago

Before current commit, sql waited for ddl operations to be prepared
in the cluster, but not committed or aborted. As a result, when the
user queries the space right after the previous ddl successfully
returned "ok", it was possible to retrieve "no space found" error.

Also fix some fresh clippy warnings

This reverts commit 5a5a1919.