Re: [I] Node SDK: wire up snapshot [iggy]

["rk-roshan-kr (via GitHub)" <gi...@apache.org>]

rk-roshan-kr commented on issue #4331:
URL: https://github.com/apache/iggy/issues/4331#issuecomment-5955749502

   Hi @justinmclean @hubcio, I would like to work on this issue. Could you please assign it to me?
   
   **Plan:**
   - Add `foreign/node/src/wire/system/snapshot.command.ts` implementing the binary wire protocol (`[compression:1][types_count:1][snapshot_type:1]*`) and handling the raw ZIP archive response.
   - Export it from `system/index.ts` and register it in `command-set.ts` under `systemAPI` next to `getStats`.
   - Remove the `@TODO` on command code 11 in `command.code.ts`.
   - Add serialization/deserialization unit tests in `snapshot.command.test.ts` and integration checks in `tcp.raw.e2e.ts`.
   
   I have the implementation, unit tests, and TypeScript builds verified locally.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org