Up to this patch `instance_uuid` and `replicaset_uuid` were generated
using name-based UUID v3. So all instances named `i1` in all clusters
had the same UUID.

With this patch UUIDs become random v4. Particular UUID values are
generated in two locations:

1. `start_boot` (src/lib.rs)
2. `handle_join_request_and_wait` -> `build_instance` (src/rpc/join.rs)

Function `Instance::new()` is removed in favor of explicit struct
construction.

The main purpose is to reduce the usage of `Instance::new()` function.
It's going to be removed soon. No behavior changes are introduced yet.

This reverts commit 2924f112.

Kurdakov Alexander authored 6 months ago and Yaroslav Dynnikov committed 5 months ago

Do not ban admin console via unix socket
after several failed attempts via `picodata connect`.

godzie44 authored 5 months ago and Georgy Moshkin committed 5 months ago

Closes #916

godzie44 authored 6 months ago and Georgy Moshkin committed 5 months ago

Closes #664

This is useful for debugging

From CI logs:

i1  | 2024-09-18 16:39:06.953 [35812] main/111/.proc_update_instance V> done waiting for applied index 8, current: 8
i1  | 2024-09-18 16:39:06.954 [35812] main/104/interactive I> self-activated successfully
i1  | 2024-09-18 16:39:06.954 [35812] main/104/interactive I> starting postgres server at ("127.0.0.1", 5734)...
i1  | CRITICAL: io error: Address already in use (os error 98)
i1  | 2024-09-18 16:39:06.954 [35812] main/104/interactive C> io error: Address already in use (os error 98)

godzie44 authored 6 months ago and Georgy Moshkin committed 5 months ago

Closes #708

godzie44 authored 6 months ago and Georgy Moshkin committed 5 months ago

fix(plugin): remove `on_config_validate` callback and introduce `ServiceRegistry::add_config_validator` function

These changes make it possible to change the configuration for a disabled plugin, as well as initiate a configuration change from instances where the plugin was not enabled.

Closes #845

Kurdakov Alexander authored 5 months ago and Alexander Kurdakov committed 5 months ago

Tier of replicaset if exists must match with tier from arguments of rpc.

godzie44 authored 6 months ago and Georgy Moshkin committed 5 months ago

Closes: #837

This commit restores the user name in the ALTER USER command to "Dave" after
it was mistakenly changed to "DAVE", a nonexistent user. This change
broke the test logic, causing the test to erroneously verify granting
privileges to a nonexistent user, rather than confirming that we can
grant default privileges. The previous implementation resulted in noop,
so this went unnoticed until we started validating user existence.

Here is the commit changing Dave to DAVE:
https://git.picodata.io/picodata/picodata/picodata/-/commit/ecbb520b559ceb7f2dbe7ae0462c8d062faf1589

Georgy Moshkin authored 6 months ago and Yaroslav Dynnikov committed 5 months ago

Strictness!