-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor archiver public API to return ArchiveBlockOutcome #3108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few things can be improved, but looks good overall
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks goodl
411a078
a1664c1
to
411a078
Compare
Done, with mass renames as requested. |
encoded_block, | ||
block_object_mappings, | ||
!sync_oracle.is_major_syncing(), | ||
) { | ||
); | ||
send_object_mapping_notification( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should skip this call if object mappings list is empty
This PR prepares for PR #3100 by moving piece mappings from
NewArchivedSegment
into a newArchiveBlockOutcome
return type. It also updates the RPCs with a notification channel for these separate mappings.As part of this change, the archiver creates
GlobalObject
s directly, which significantly simplifies the code.Code contributor checklist: