apollo_consensus_orchestrator: add Deserialize for testing#13945
Conversation
PR SummaryLow Risk Overview In Reviewed by Cursor Bugbot for commit dd4db61. Bugbot is set up for automated code reviews on this repo. Configure here. |
1c25347 to
5707677
Compare
5707677 to
dd4db61
Compare
yoavGrs
left a comment
There was a problem hiding this comment.
@yoavGrs made 1 comment.
Reviewable status: 0 of 6 files reviewed, 1 unresolved discussion (waiting on dorimedini-starkware and matanl-starkware).
a discussion (no related file):
In general, why is deserialization gated? For heavy objects?
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware made 1 comment.
Reviewable status: 0 of 6 files reviewed, 1 unresolved discussion (waiting on matanl-starkware and yoavGrs).
a discussion (no related file):
Previously, yoavGrs wrote…
In general, why is deserialization gated? For heavy objects?
because it's only needed in my test. if it ever has a real use case, we can remove the gate
yoavGrs
left a comment
There was a problem hiding this comment.
@yoavGrs reviewed 6 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on matanl-starkware).

No description provided.