Skip to content

fix(*): rollback when target chain is not available#201

Merged
jzhe886 merged 1 commit intorelease-1.6from
fix/rollback-index-1.6
May 21, 2021
Merged

fix(*): rollback when target chain is not available#201
jzhe886 merged 1 commit intorelease-1.6from
fix/rollback-index-1.6

Conversation

@Alexader
Copy link
Contributor

No description provided.

@Alexader Alexader force-pushed the fix/rollback-index-1.6 branch from 3e127fa to d94d7fb Compare May 17, 2021 17:13
@codecov-commenter
Copy link

Codecov Report

Merging #201 (d94d7fb) into release-1.6 (8ad5cfc) will decrease coverage by 1.73%.
The diff coverage is 66.07%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.6     #201      +/-   ##
===============================================
- Coverage        73.08%   71.35%   -1.74%     
===============================================
  Files               33       34       +1     
  Lines             2222     2388     +166     
===============================================
+ Hits              1624     1704      +80     
- Misses             404      479      +75     
- Partials           194      205      +11     
Impacted Files Coverage Δ
internal/agent/client.go 63.41% <0.00%> (-1.05%) ⬇️
pkg/plugins/grpc.go 73.45% <0.00%> (-4.41%) ⬇️
internal/executor/handle.go 62.36% <33.33%> (-18.24%) ⬇️
internal/exchanger/exchanger.go 75.74% <53.65%> (-3.50%) ⬇️
internal/syncer/syncer.go 74.44% <60.00%> (-1.10%) ⬇️
internal/syncer/persist.go 57.14% <66.66%> (+4.20%) ⬆️
internal/syncer/agent.go 74.62% <67.64%> (-1.24%) ⬇️
internal/exchanger/handler.go 63.92% <72.22%> (-4.11%) ⬇️
internal/exchanger/receipt.go 86.36% <86.36%> (ø)
internal/exchanger/recover.go 84.61% <92.59%> (+2.26%) ⬆️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ad5cfc...d94d7fb. Read the comment docs.

@Alexader Alexader force-pushed the fix/rollback-index-1.6 branch 3 times, most recently from 0341aaf to 5c2228b Compare May 18, 2021 09:47
Comment on lines +11 to +14
var pool *Pool
act, loaded := ex.receipts.Load(receipt.Ibtp.To)
if !loaded {
pool = NewPool()
ex.receipts.Store(receipt.Ibtp.To, pool)
} else {
pool = act.(*Pool)
}
pool.feed(receipt)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace with loadorstore()

@Alexader Alexader force-pushed the fix/rollback-index-1.6 branch from 5c2228b to 9c13274 Compare May 20, 2021 08:05
@jzhe886 jzhe886 merged commit 8d397e8 into release-1.6 May 21, 2021
@jzhe886 jzhe886 deleted the fix/rollback-index-1.6 branch May 21, 2021 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants