numinnex opened a new issue, #2590: URL: https://github.com/apache/iggy/issues/2590
We have an Identifier struct, that we use to store both numeric as well as string variant of id's. SDK user can use the Identifier to name resources such as streams/topics/consumer_groups. Because of our internal logic, we have to find a way to make it such that the `Identifier` struct does not reach the `StateMachine` state of command application (state machines are isolated and they can't reach to indexes of identifiers of other states -- example `ConsumerGroup` state needs to know about stream_id and topic_id, but does not have access to `Streams` state indexes to perform the tanslation Identifier -> usize). I propose that we move the identifier translation step somewhere near the command parsing step, so the moment the command is fully parsed and we enter the internals of our system, we work only with numeric ids, rather than the Identifier union. Please, leave a comment if you'd want to pick up the issue as it's part of the current ongoing work on cluster, which touches crates that aren't part of the `server`. -- 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]
