matanper opened a new pull request, #3191: URL: https://github.com/apache/iggy/pull/3191
Expose the delete segments command in the Go SDK so clients can manage partition segment cleanup over TCP. ## Which issue does this PR close? Closes #3190 ## Rationale The Go SDK was missing support for the existing `DeleteSegments` TCP command, so Go clients could not trigger partition segment cleanup through the SDK. ## What changed? The Go SDK now exposes `DeleteSegments(streamId, topicId, partitionId, segmentsCount)`, adds the TCP command code and binary payload serialization, and includes a serialization test for the command wire format. ## Local Execution - Passed: `go test ./...` from `foreign/go` - Passed: live Go SDK smoke test against `apache/iggy:0.8.0` on TCP, including login, stream/topic creation, message send, `DeleteSegments`, topic fetch, and cleanup - Pre-commit hooks not ran ## AI Usage <!-- If AI tools were used, please answer: 1. Which tools? (e.g., GitHub Copilot, Claude, ChatGPT) 2. Scope of usage? (e.g., autocomplete, generated functions, entire implementation) 3. How did you verify the generated code works correctly? 4. Can you explain every line of the code if asked? If no AI tools were used, write "None" or delete this section. --> 1. Cursor 2. Entire implementation 3. Ran against apache/iggy:0.8.0 and tested it live 4. Yes -- 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]
