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

Added a multiplicity test example #254

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rekhoff
Copy link
Contributor

@rekhoff rekhoff commented Feb 21, 2025

Creates a small sample program with a Rust server and C# client that tests various subscriptions and outputs the results to the CLI.

Description of Changes

This is a new example project that is used to test subscription multiplicity.
It currently tests for 2 scenarios:

  1. A single subscription handle, using an array of subscription query with overlapping content against a single table.
    Data is modified on the server that results in both additions and removal of items from the scope of subscribed data, and the results are validated.

  2. 2 subscription handles, each with a single subscription query against a single table, with overlapping content.
    Data is modified on the server that results in both additions and removal of items from the scope of subscribed data, and the results are validated. One of the subscription handles is unsubscribed to, and the results are validated.

Building the Rust server and then running the C# client from CLI will output the tests and results to the terminal.

API

  • This is an API breaking change to the SDK

Requires SpacetimeDB PRs

SpacetimeDB: jsdt/multi-subscribe
com.clockworklabs.spacetimedbsdk: jgilles/unsubscribe-fix

Testsuite

SpacetimeDB branch name: jsdt/multi-subscribe

Testing

This example project is a test itself.

  • Ran the C# client and read output results, confirming expected behavior.

kazimuth and others added 2 commits February 21, 2025 10:15
Warning message fix (thanks Jeff)

Update for new Subscribe/UnsubscribeMulti

thanks zeke

Add MultiDictionary; update dispatching logic still wrong

Add even more wacky data structures

WHOOOOOO

Fix MultiDictionary bug

More asserts and comments

Tests passing

Better errors, purge dead package

Test

Update for blackholio

Update blackholio

Fix comment

Remove dead

One more dead

COMMENT

More comments

Added a multiplicity test example

Creates a small sample program with a Rust server and C# client that tests various subscriptions and outputs the results to the CLI.

Make sure we insert before we delete

Fix more comments

Add CRDT test for MultiDictionaryDelta

Revert "Added a multiplicity test example"

This reverts commit ff10925.

Undo DLL changes
Creates a small sample program with a Rust server and C# client that tests various subscriptions and outputs the results to the CLI.
@rekhoff rekhoff self-assigned this Feb 21, 2025
Base automatically changed from jgilles/unsubscribe-fix to master February 25, 2025 21:24
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