Skip to content
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

Improve cross2eth stability 0224 #1161

Merged
merged 21 commits into from
Mar 1, 2022

Conversation

zhengjunhe
Copy link
Collaborator

1.增加chain33发送到ethereum中继交易的跟踪信息保存在数据库中,需要得到ack信息,否则定期重发;
2.增加ethereum发送到chain33中继交易的跟踪信息保存在数据库中,需要得到ack信息,否则定期重发;
3.通过增加以上2种send,ack的方式保证中继服务的稳定性,即使在出现网络错误,中继panic的情况下,重启,而不影响中继服务的可靠性;同时加入的不同的checkpoint的日志,用于跟踪一个交易的不同点的过程,可以帮助后续定位问题,解决问题,完善实现机制,进一步提高可靠性、稳定性。

@codecov
Copy link

codecov bot commented Feb 28, 2022

Codecov Report

Merging #1161 (b3d8752) into master (8b20a36) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head b3d8752 differs from pull request most recent head 0ebe48b. Consider uploading reports for the commit 0ebe48b to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1161   +/-   ##
=======================================
  Coverage   57.74%   57.75%           
=======================================
  Files         212      212           
  Lines       33939    33939           
=======================================
+ Hits        19599    19601    +2     
+ Misses      12625    12624    -1     
+ Partials     1715     1714    -1     
Impacted Files Coverage Δ
plugin/store/mpt/db/trie.go 55.16% <0.00%> (-0.59%) ⬇️
plugin/store/mpt/db2/trie.go 55.16% <0.00%> (-0.59%) ⬇️
plugin/p2p/gossip/p2pserver.go 49.06% <0.00%> (-0.47%) ⬇️
plugin/consensus/qbft/peer_set.go 63.78% <0.00%> (+0.23%) ⬆️
plugin/consensus/qbft/consensus_state.go 60.32% <0.00%> (+0.28%) ⬆️
plugin/p2p/gossip/tls.go 28.12% <0.00%> (+1.87%) ⬆️

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 8b20a36...0ebe48b. Read the comment docs.

@YingQm YingQm closed this Feb 28, 2022
@YingQm YingQm reopened this Feb 28, 2022
@zhengjunhe
Copy link
Collaborator Author

关闭重新触发ci

@zhengjunhe zhengjunhe closed this Mar 1, 2022
@zhengjunhe zhengjunhe reopened this Mar 1, 2022
@vipwzw vipwzw merged commit cac69c7 into 33cn:master Mar 1, 2022
@33cn
Copy link
Owner

33cn commented Mar 21, 2022

🎉 This PR is included in version 1.67.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@33cn 33cn added the released label Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants