Skip to content

ali/ssubscribe#1

Open
atakavci wants to merge 7 commits intomainfrom
ali/ssubscribe
Open

ali/ssubscribe#1
atakavci wants to merge 7 commits intomainfrom
ali/ssubscribe

Conversation

@atakavci
Copy link
Copy Markdown
Owner

No description provided.

vandyvilla and others added 3 commits May 20, 2025 10:19
* Support sharded pubsub

* Support sharded pubsub

* fix api

* fix enum

* fix api

---------

Co-authored-by: Marc Gravell <marc.gravell@gmail.com>
* Support sharded pubsub

* Support sharded pubsub

* fix api

* fix enum

* fix api

* proposed refactor to single-field implementation

* RedisDatabase needs to use SPUBLISH/PUBLISH correctly

* support ssub on CLIENT response

* add to features; fixup shipped etc

---------

Co-authored-by: xli <xli@roblox.com>
@atakavci atakavci force-pushed the ali/ssubscribe branch 3 times, most recently from 841fdc6 to 65709ac Compare May 30, 2025 15:01
atakavci pushed a commit that referenced this pull request Mar 18, 2026
* minimal pre-v3 consolidation:

- add RESPite to the solution, but *DO NOT* reference from SE.Redis
- update eng tooling and migrate field parsing from FastHash to AsciiHash
- update the toy server *completely* to the v3 specification

* - backport LCS and RedisType.VectorSet changes from v3

* CI

* tidy TFMs

* nix TestHarness

* deal with TFM-specific warnings from .NET 10

* in-proc test enhancements

* fix Select

* fix "fromBytes" logic to be more formal, and more importantly: to keep Nick happy

* make DebugAssertValid logic #if DEBUG

* cleanup a great many #if

* join NET7 => NET8

* join net9 with net10

* fix high integrity token check on AUTH/HELLO

* borked a #endif

* Use a better way of refing the build project

* clarify public API files

* exclude Build.csproj from refing the analyzer project!

* - improve "moved" test for "did the client already know the new node?"
- fix where RESP3 default is specified for the toy server

* make test server output more terse

* toy server: don't pretend SENTINEL exists

* rev minor

* bump

* Include endpoint data in log, for example `[127.0.0.1:6379 #1] CLIENT => +OK`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants