-
Notifications
You must be signed in to change notification settings - Fork 366
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
INDY-1740: Fixed a bug in re-asking cancellation #933
INDY-1740: Fixed a bug in re-asking cancellation #933
Conversation
spivachuk
commented
Oct 3, 2018
- Corrected the test of cancellation of scheduled re-asking for ledger statuses and maximal consistency proofs on reception of sufficient count of messages.
- Fixed cancellation of scheduled re-asking for ledger statuses and maximal consistency proofs which was not operable previously.
- Corrected the test of cancellation of scheduled re-asking for ledger statuses and maximal consistency proofs on reception of sufficient count of messages. Signed-off-by: Nikita Spivachuk <nikita.spivachuk@dsr-company.com>
- Fixed a bug in cancellation of scheduled re-asking for ledger statuses and maximal consistency proofs. Signed-off-by: Nikita Spivachuk <nikita.spivachuk@dsr-company.com>
ledgerInfo = self.getLedgerInfoByType(ledger_id) | ||
nodes = [node for node in self.owner.nodeReg if node not in ledgerInfo.ledgerStatusOk] | ||
self.owner.request_ledger_status_from_nodes(ledger_id, nodes) | ||
|
||
def reask_for_last_consistency_proof(self, ledger_id): | ||
if ledger_id in self.request_consistency_proof_action_ids: |
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.
Is this condition really not needed?
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.
Made entry removal safe.
|
||
def reask_for_ledger_status(self, ledger_id): | ||
if ledger_id in self.request_ledger_status_action_ids: |
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.
Is this condition really not needed?
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.
Made entry removal safe.
- Corrected removal of action IDs from LedgerManager.request_ledger_status_action_ids and LedgerManager.request_consistency_proof_action_ids. Signed-off-by: Nikita Spivachuk <nikita.spivachuk@dsr-company.com>