numinnex commented on PR #2887:
URL: https://github.com/apache/iggy/pull/2887#issuecomment-4022907482

   Since the `Command` exists in our header frame, early validation of that 
should be trivial as we just need to get the 
bytes[command_bytes_start..command_bytes_end]. the tricky part is `operation` 
as this enum has to be lazily validated, it means that we have to eagerly 
construct the header and then validate whether the value represents correct 
state, thus the `Operation` enum needs to become a "catch-all" enum.


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