diff options
| author | Jimmy Hu <jimmy.hu@dexon.org> | 2019-05-02 20:35:18 +0800 |
|---|---|---|
| committer | Jimmy Hu <jimmy.hu@dexon.org> | 2019-05-02 21:04:23 +0800 |
| commit | 1edd55a2ff91bbbe94b8c1e8564ea2e5a45cf37c (patch) | |
| tree | e005aadd788f93806335669aa720b83cf4d84038 /core/syncer/agreement.go | |
| parent | 86befa5015417359f7b569cff9d6c3f02045f394 (diff) | |
| download | dexon-consensus-testnet.tar.gz dexon-consensus-testnet.tar.zst dexon-consensus-testnet.zip | |
core: optimize syncer handling agreement resulttestnet
Diffstat (limited to 'core/syncer/agreement.go')
| -rw-r--r-- | core/syncer/agreement.go | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/core/syncer/agreement.go b/core/syncer/agreement.go index b414e11..d39c246 100644 --- a/core/syncer/agreement.go +++ b/core/syncer/agreement.go @@ -176,12 +176,7 @@ func (a *agreement) processAgreementResult(r *types.AgreementResult) { a.logger.Trace("Agreement result cached", "result", r) return } - notarySet, err := a.cache.GetNotarySet(r.Position.Round) - if err != nil { - a.logger.Error("unable to get notary set", "result", r, "error", err) - return - } - if err := core.VerifyAgreementResult(r, notarySet); err != nil { + if err := core.VerifyAgreementResult(r, a.cache); err != nil { a.logger.Error("Agreement result verification failed", "result", r, "error", err) @@ -257,18 +252,13 @@ func (a *agreement) processNewCRS(round uint64) { a.latestCRSRound = round // Verify all pending results. for r := prevRound; r <= a.latestCRSRound; r++ { - notarySet, err := a.cache.GetNotarySet(r) - if err != nil { - a.logger.Error("Unable to get notary set", "round", r, "error", err) - continue - } pendingsForRound := a.pendingAgrs[r] if pendingsForRound == nil { continue } delete(a.pendingAgrs, r) for _, res := range pendingsForRound { - if err := core.VerifyAgreementResult(res, notarySet); err != nil { + if err := core.VerifyAgreementResult(res, a.cache); err != nil { a.logger.Error("Invalid agreement result", "result", res, "error", err) |