atharvalade commented on PR #2916:
URL: https://github.com/apache/iggy/pull/2916#issuecomment-4040937171
> The `FillSnapShot` trait bound has become a bit viral. I'm thinking of
ways to reduce that.
hmm maybe you can try a blanket super-trait alias like `trait MetadataStm:
StateMachine<...> + FillSnapshot<MetadataSnapshot> {}` with a blanket impl. It
won't reduce the actual propagation but cuts the noise from 4 lines to 1 at
each use site. Longer term, pulling checkpoint into a separate component that
owns the `FillSnapshot` concern behind a callback would remove the bound from
IggyMetadata's `M` parameter entirely.
--
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]
