-
Notifications
You must be signed in to change notification settings - Fork 685
Implement server-side support for the v2 websocket protocol #4213
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
Merged
Merged
Changes from all commits
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
ebed4c3
Split WS into v1 and v2; make various changes in v2.
gefjon 4fc9dc1
Merge remote-tracking branch 'origin/master' into phoebe/websocket-v2
gefjon 7a1258a
Fix some copy-paste typos
gefjon 05478c8
Nix `NoSuccessNotify`, add doc comments
gefjon 4a4b10b
Update references to v1 WS format
gefjon dd020e6
Add doc comment to `TableUpdateRows`
gefjon e4e46c0
Fix compilation issue
jsdt f28f81e
Move compression into common and add new protocol option for v2
jsdt b621b2a
WIP on new subscription update format
jsdt 8ca7795
WIP on subscription update stuff, and add UnsubscribeFlags
jsdt 89a09c2
Move BsatnRowList to common and use it in v2
jsdt d9e9bfe
Update module subscription manager to group updates by client/query.
jsdt 9437a55
Merge origin/master into jsdt/ws-v2
cloutiertyler 0906a38
Derive Ord for TableName and ReducerName
cloutiertyler 6a68b54
Fix Rust SDK import of RowListLen from common module
cloutiertyler aaa8b97
fix a issue from having missing files when merging
jsdt 34328a8
Plumb reducer return value
jsdt a21b07d
finish send worker for v2
jsdt 39dac16
send reducer result even if the client isn't getting something
jsdt bc1ebdd
More subscription manager
jsdt 760379e
Cleanup
jsdt d9a8b0c
Merge branch 'master' into jsdt/ws-v2
jsdt b53b97c
Fix reducer return issue
jsdt b5b3757
tiny test
jsdt 734f370
remove on errors
jsdt 46d01fe
Add a simple test for removal
jsdt e27f45b
more add v2 sub
jsdt f4c73ca
One off queries and call procedure
jsdt 7bdbb52
Add more one off query stuff
jsdt 3571b98
fmt
jsdt 516b0af
Send compression tags
jsdt 3d3d3ef
Tweak v2 serialization
jsdt c14e5ed
Remove v2 subscriptions
jsdt 25017c0
Don't filter empty ranges in `BsatnRowList`
gefjon db91218
send reducer errors
jsdt 302de17
fix lint issues
jsdt 4f2eb9d
Treat unhandled errors as fatal for v2 clients
jsdt 9444733
Don't send any reducer info for v10 modules
jsdt ded3e23
Add back some metrics for v2
jsdt 5c06e8a
Merge branch 'master' into jsdt/ws-v2
jsdt dd12852
Undo some unintentional changes
jsdt 0eeb507
Merge branch 'master' into jsdt/ws-v2
jsdt 1bdea5c
Remove reducer flags
jsdt 3d6c6bf
Expand on a comment
jsdt ec84eb1
Revert "Remove reducer flags"
jsdt 85681e5
clean up some match cases
jsdt ad9ca8b
move compression tags to common, fix metrics issue
jsdt 67b8621
Comments and fix typo
jsdt 228765a
Don't merge QueryRows
jsdt f342b51
Import compression from common
jsdt b83f26d
Bring some optimizations to v2 encoding
jsdt 51363d5
Add/update some comments
jsdt 5203cec
Fix lint
jsdt 3ca35b3
Merge branch 'master' into jsdt/ws-v2
jsdt 238b5c3
Refactor error sending into a helper
jsdt a616d60
Merge branch 'master' into jsdt/ws-v2
jsdt cde0b6b
Take view bindings from master
jsdt dabf177
Comment out the check diff for csharp
jsdt 2d04f80
Change some warning to errors
jsdt 6527dba
Merge branch 'master' into jsdt/ws-v2
jsdt 149234a
Merge branch 'master' into jsdt/ws-v2
jsdt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.