Skip to content
This repository has been archived by the owner on Sep 13, 2022. It is now read-only.

change(network): output more info for debug rpc timeout #157

Merged
merged 5 commits into from
Feb 10, 2020

Conversation

zeroqn
Copy link
Contributor

@zeroqn zeroqn commented Feb 10, 2020

What type of PR is this?
change

What this PR does / why we need it:

  • output connected address with rpc timeout and drop error
  • output connected peers number when poll peer manager
  • delete rpc entry in rpc map after timeout

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@yejiayu yejiayu merged commit abc3637 into nervosnetwork:master Feb 10, 2020
zeroqn added a commit to zeroqn/muta that referenced this pull request Feb 11, 2020
…k#157)

* feat(network): output peer connected addr on RPC error

* change(network): also output number of connected peers

* change(network): remove rpc entry after timeout

* doc(network): comment about expect() on Option session public key

* change(network): output more info on rpc entry not found
zeroqn added a commit to zeroqn/muta that referenced this pull request Feb 11, 2020
…k#157)

* feat(network): output peer connected addr on RPC error

* change(network): also output number of connected peers

* change(network): remove rpc entry after timeout

* doc(network): comment about expect() on Option session public key

* change(network): output more info on rpc entry not found
cryptowen pushed a commit that referenced this pull request Feb 12, 2020
* bench excute (#145)

* fix(makefile): Docker push to the correct image (#146)

* feat: Update juniper, supports async (#149)

* feat: Update juniper

* fix ci

* fix ci

* fix(network): give up retry (#152)

* change(network): only tag unconnectable for io::ErrorKind::Other

* fix(network): never reconnect later

* fix(network): clippy warning

* flush mempool after storage (#155)

* feat: add common_ref argument for keypair tool (#154)

* feat: add common_ref arg for keypair tool

* new line

* feat: add panic hook to logger (#156)

* feat: add panic hook to logger

* chore: change crate location

* filter callback_cache while pull txs (#158)

* change(network): output more info for debug rpc timeout (#157)

* feat(network): output peer connected addr on RPC error

* change(network): also output number of connected peers

* change(network): remove rpc entry after timeout

* doc(network): comment about expect() on Option session public key

* change(network): output more info on rpc entry not found

* feat: update overlord with brake engine (#159)

* feat(consensus): update overlord with brake engine

* chore: update overlord version

* feat: register choke handler in p2p

* update overlord

tmp

* feat(consensus): update overlord with brake engine

* chore: update overlord version

* feat: register choke handler in p2p

* update overlord

tmp

update overlord

update overlord

update overlord

update overlord

* update overlord

update overlord

* fix(sync): Save the latest height when an error occurred

* fix(sync): Check remote block

* chore: Add log for check block

* chore: print order hashes length

* feat: add panic hook to logger (#156)

* feat: add panic hook to logger

* chore: change crate location

* refactor: Remove boradcast when start sync

* fix: empty cycle used in genesis block

* filter callback_cache while pull txs (#158)

* chore: update overlord

* fix CI

Co-authored-by: Jiayu Ye <yejiayu.fe@gmail.com>
Co-authored-by: Wenchao Hu <me@huwenchao.com>
Co-authored-by: rev-chaos <wancencen@cryptape.com>

* fix(doc): fix graphql_api doc link and doc-api build sh (#161)

* fix(metadata): test broken after consensus update

Co-authored-by: rev-chaos <wancencen@cryptape.com>
Co-authored-by: Jiayu Ye <yejiayu.fe@gmail.com>
Co-authored-by: Wenchao Hu <me@huwenchao.com>
Co-authored-by: Eason Gao <kaoimin@qq.com>
Co-authored-by: LycrusHamster <liang2929@hotmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants