Skip to content

feat(js): implement new zero copy message format for send & poll#1757

Merged
spetz merged 2 commits into
apache:masterfrom
T1B0:sendmessage-new-zc
May 13, 2025
Merged

feat(js): implement new zero copy message format for send & poll#1757
spetz merged 2 commits into
apache:masterfrom
T1B0:sendmessage-new-zc

Conversation

@T1B0
Copy link
Copy Markdown
Contributor

@T1B0 T1B0 commented May 8, 2025

/!\ WIP: DO NOT MERGE /!\

debugging new sendMessage serialization format

@T1B0 T1B0 force-pushed the sendmessage-new-zc branch from c0123a7 to 06682da Compare May 8, 2025 22:22
@T1B0 T1B0 changed the title Draft: start implement new zero copy sendMessage format (s01e01 debug) feat: implement new zero copy message format for send & poll May 8, 2025
@T1B0
Copy link
Copy Markdown
Contributor Author

T1B0 commented May 8, 2025

seems sendMessage & poll works now

@numinnex
Copy link
Copy Markdown
Contributor

Hey @T1B0, thanks for updating the node SDK, does that mean that we can merge this PR ?

@T1B0 T1B0 force-pushed the sendmessage-new-zc branch from 06682da to 0649e4f Compare May 12, 2025 17:29
@T1B0
Copy link
Copy Markdown
Contributor Author

T1B0 commented May 12, 2025

Thanks for noticing this one @numinnex ! i just rebased it, brings support to new zero-copy serialization format so yes let's merge it please so we support latest server protocol version 👍 cc @hubcio @spetz

@T1B0 T1B0 force-pushed the sendmessage-new-zc branch from 0649e4f to 0c2c712 Compare May 12, 2025 18:51
@T1B0 T1B0 changed the title feat: implement new zero copy message format for send & poll feat(,: implement new zero copy message format for send & poll May 12, 2025
@T1B0 T1B0 changed the title feat(,: implement new zero copy message format for send & poll feat(js): implement new zero copy message format for send & poll May 12, 2025
@T1B0 T1B0 force-pushed the sendmessage-new-zc branch from 0c2c712 to 38a6bee Compare May 13, 2025 05:39
@spetz spetz merged commit 22fd92c into apache:master May 13, 2025
27 checks passed
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.

4 participants