justinmclean opened a new issue, #4331: URL: https://github.com/apache/iggy/issues/4331
The Rust client has `SystemClient::snapshot` (`core/common/src/traits/system_client.rs`), which returns a compressed snapshot of server state for debugging. It is the only client call the Node SDK does not have. The command code is already in `foreign/node/src/wire/command.code.ts` (`GetSnapshot: 11`, marked `@TODO`), but nothing uses it. ## What to do - Add `foreign/node/src/wire/system/snapshot.command.ts`, following `get-stats.command.ts` in the same folder. The request and response formats are in `core/binary_protocol/src/requests/system/get_snapshot.rs` and `core/binary_protocol/src/responses/system/get_snapshot.rs`. - Export it from `system/index.ts` and register it in `foreign/node/src/wire/command-set.ts` next to `getStats`. - Remove the `@TODO` on the `GetSnapshot` entry. - Add a unit test for the serialisation, and an e2e test in `foreign/node/src/e2e/tcp.raw.e2e.ts` next to the `getStats` one. ## Done when `snapshot` returns a non-empty result against a running server, and the new tests pass. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
