Add/accountsync#77
Merged
Merged
Conversation
Introduce the AccountSyncRequestAccounts message to facilitate stateless targeted fetch requests for accounts. This message includes a map of addresses and a phase field, enhancing the account synchronization process. Update related protobuf definitions to maintain consistency across the protocol.
Implement the HandleAccountsFetch method in the Datarouter to process account synchronization requests for specific addresses. Introduce a new helper function, FetchAccountsByAddresses, to retrieve account data based on provided addresses. Update constants to include new request and response phases for account synchronization. This enhances the account synchronization process by allowing targeted fetching of account information.
Implement the FetchAccounts method in AccountSync to send targeted requests for specific accounts using the new AccountsSyncFetchProtocol. Update the Communicator interface and add necessary handlers in the Sync protocol to support this functionality. Enhance the accounts fetching process by ensuring proper error handling and logging. This update improves the overall account synchronization capabilities.
Update the Reconcile method in the Reconciliation router to accept an additional parameter for remote availability responses. Implement a new phase to pre-create accounts missing from the local database by fetching them from the remote source. This improves the account reconciliation process by ensuring that all relevant accounts are accounted for, even if they were not previously present in the local database.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.