Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(server): Replace channel slice with store #938

Merged
merged 4 commits into from
Mar 20, 2023

Conversation

dranikpg
Copy link
Contributor

No description provided.

@dranikpg dranikpg requested a review from romange March 13, 2023 21:28
@dranikpg dranikpg force-pushed the new-pubsub branch 2 times, most recently from d8edc81 to 29a3d92 Compare March 14, 2023 13:20
@dranikpg dranikpg marked this pull request as ready for review March 14, 2023 13:20
src/server/server_family.h Show resolved Hide resolved
src/server/channel_store.h Outdated Show resolved Hide resolved
src/server/channel_store.h Outdated Show resolved Hide resolved
src/server/channel_store.h Outdated Show resolved Hide resolved
Signed-off-by: Vladislav Oleshko <[email protected]>
Signed-off-by: Vladislav Oleshko <[email protected]>
@dranikpg dranikpg requested a review from romange March 18, 2023 20:02
@romange
Copy link
Collaborator

romange commented Mar 19, 2023

how did you test it works? btw, Pascal submitted a docker that tests his client library against DF.

@dranikpg
Copy link
Contributor Author

I checked them, I ran the regression tests, I checked manually

@romange romange merged commit 03fb21a into dragonflydb:main Mar 20, 2023
@dranikpg dranikpg deleted the new-pubsub branch March 22, 2023 07:05
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.

2 participants