haubur opened a new issue, #3881:
URL: https://github.com/apache/iggy/issues/3881

   ### Bug description
   
   [IggyConsumer::delete_offset() 
](https://github.com/apache/iggy/blob/8890888228fab0cb9cb9abe1691c8a6be6c90435/core/sdk/src/clients/consumer.rs#L283)
 takes an optional `partition_id`.
   
   What happens for `partition_id = None`:
   - For a consumer group member the server resolves which partition the 
consumer is currently assigned to.
   The partition is correctly inferred.
   - For a standalone consumer  `partition_id = None` always deletes the offset 
in partition 0. Not the assigned one.
   
   I have a test here that proves that: 
https://github.com/haubur/iggy/blob/fix/delete-offset/core/integration/tests/sdk/consumer_offset.rs
   
   The solution would be to resolve `None` into the actual assigned 
`partition_id` just like `store_offset()` does here: 
https://github.com/apache/iggy/blob/8890888228fab0cb9cb9abe1691c8a6be6c90435/core/sdk/src/clients/consumer.rs#L260
   
   ### Affected area / component
   
   Rust SDK
   
   ### Deployment
   
   None
   
   ### Versions
   
   _No response_
   
   ### Hardware / environment
   
   _No response_
   
   ### Sample code
   
   _No response_
   
   ### Logs
   
   _No response_
   
   ### Iggy server config
   
   _No response_
   
   ### Reproduction
   
   1. Checkout: 
https://github.com/apache/iggy/compare/master...haubur:iggy:fix/delete-offset
   2. `cargo test -p integration -- sdk::consumer_offset`
   
   ### Contribution
   
   - [x] I'm willing to submit a pull request to fix this bug
   
   ### Good first issue
   
   - [ ] I think this could be a good first issue for a new contributor


-- 
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]

Reply via email to