Madelyn Olson

Move to C11 and adopt _Atomic, static_asserts, and thread_local

3

As per https://github.com/valkey-io/valkey/commit/445a4b669a3a7232a18bf23340c5f7d580aa92c7, we implemented custom logic to compile with C11 when atomics were available, and C99 when they were not available. However, with the EoL of CentOS 7, we...

MacOS 11 runners are being deprecated by June 2024, we should update our reference to macOS 12. See: https://github.com/valkey-io/valkey/blob/e7e5a104ec9b3274577dce9cf1a35ea8b8f8e008/.github/workflows/daily.yml#L899 We should probably also update `macos-13` -> `macos-14`, as `macos-latest` now...

[NEW] Compact variant CLUSTER SLOTS DENSE

17

One recurring issue that we, at AWS, have noticed is that over time clusters will naturally have slot ranges become fragmented between primaries. For example, if you have 2 nodes,...

Replace CentOS 7 image with CentOS Stream 9

1

CentOS 7 is EoL, we should replace it with the replacement which is CentOS Stream. CentOS Stream is about to be EoL https://blog.centos.org/2023/04/end-dates-are-coming-for-centos-stream-8-and-centos-linux-7/. The official container image is being distributed...

Validate format of YAML files

2

Our YAML files for our github actions use inconsistent format, and it would be nice to have them aligned. In line with the our other discussions, we should add a...

There are quite a few places where we call [connSetReadHandler](https://github.com/search?q=repo%3Avalkey-io%2Fvalkey%20connSetReadHandler&type=code) and [connSetWriteHandler](https://github.com/search?q=repo%3Avalkey-io%2Fvalkey%20connSetWriteHandler&type=code) and don't check the return code. From an API perspective, we only need to check the return code...

This pierces the connection abstraction, which we don't want to do. Connections are supposed to be opaque to us. This is also documented clearly enough to understand why we are...

Migrate TLS.md into README.md

2

While reviewing some old files, we noticed https://github.com/valkey-io/valkey/blob/unstable/TLS.md should probably be in the main readme instead of a separate file. There are also two todo items, we should get if...

[NEW] Add support for setting the group on a unix domain socket

1

From a mastodon conversation. ``` By default the socket is created in the wheel group, which I clearly don’t want to add my application user use to, so I checked...

Followup items from #758 Async IO threads

16

List of pending items from #758, all of these are future enhancements, and not pending issues that need to be merged before Valkey 8 (but they could be). - [x]...