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

speedup import block processing #1093

Closed
wants to merge 9 commits into from
Closed

Conversation

laizy
Copy link
Contributor

@laizy laizy commented Oct 11, 2019

No description provided.

@laizy laizy closed this Oct 11, 2019
@laizy laizy reopened this Oct 11, 2019
rongyi and others added 7 commits October 17, 2019 11:45
* fixed int128::min() loose precion bug

* Update vm/neovm/params_builder.go

Co-Authored-By: laizy <[email protected]>
* update ontid with new features:

1. controller
2. new recovery design

* update events and fix bugs

* add methods of revoking ONT ID

* update GetDDO and registered methods

* update delete attributes

* fix bug in getController

* add copyrights

* fix bugs and add tests

* add copyright

* update serialization

* update checkWitness in ontid

* fix bugs

* fix bugs in using revoked key

* error handling

* add limitations

* limit attribute number

* update

* change limit to owner keys

* fix bugs

* limit total size of owner's keys
* save empty block

* handle empty block

* clear code

* fix merkletreeroot check

* fix some bug

* fix emptyblockamount bug

* fix some bug
if err != nil {
return fmt.Errorf("read block data height:%d error:%s", i, err)
var readErr error
blockes := make(chan *types.Block, 10)
Copy link
Member

Choose a reason for hiding this comment

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

blocks?
or blockC?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

replace and fixed in pr- #1112

@laizy
Copy link
Contributor Author

laizy commented Oct 28, 2019

replaced by #1112 to change branch back to master

@laizy laizy closed this Oct 28, 2019
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.

6 participants