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

opt isProposer and isEndorser #1285

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

zhiqiangxu
Copy link
Contributor

No description provided.

laizy and others added 10 commits August 5, 2020 10:25
* implement offline witness in p2p

* refactor local rpc

* add rpc api

* add license

* fix comments

* check self is gov node before vote
* remove old gov node ip from members in subnet

* fix testcase
* Drop duplicate import

* Drop redundant return and break statement

* for true ==> for

* Fix bool cmp lint

should omit comparison to bool constant

* use or drop unused variables

* Drop empty branch

* Some fmt.Sprintf is useless, drop it

* should call self.quitWg.Add(1) before starting the goroutine to avoid a race

* drop for, using append

* Update cmd/tx_cmd.go

* Update consensus/vbft/node_sync.go

Co-authored-by: laizy <laizy@users.noreply.github.com>
@laizy laizy closed this Oct 27, 2020
@laizy laizy reopened this Oct 27, 2020
@zhiqiangxu zhiqiangxu changed the title opt isProposer and isEndorSer opt isProposer and isEndorser Oct 27, 2020
@laizy laizy force-pushed the master branch 2 times, most recently from 8c8ee59 to dc599e9 Compare July 6, 2021 07:08
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.

5 participants