Skip to content

Bump the version to 2.2.0 #1853

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

Merged
merged 46 commits into from
Nov 11, 2019
Merged

Conversation

foriequal0
Copy link
Contributor

Release note:

remagpie and others added 30 commits November 8, 2019 19:18
VoteCollector is changed to return Result<bool, DoubleVote>. Rust
compiler will force you to check whether there was a double vote.
Extract the crypto module to a new repository
If generating a block takes too much time, the view could be changed
before the miner module requests the signature. If the Tendermint
module receives a signature request of an old view, it should ignore
the message. Before this commit, the Tendermint module was crashing
when it gets a signature request from an old view.
… time

The Tendermint module was changing its step to 'Propose' when the
"engine timeout empty proposal" timer is fired. If the timer is called
in the wrong time the step should not be changed. This commit makes
the step change when it is needed.
It is hard to know whether submit_proposal_block changes step or
not. Since submit_proposal_block has only two lines of code, it
doesn't remove code duplication. To clarify step transition, inline
the submit_proposal_block function.
url 1.7.0 which CodeChain used, has a lifetime bug.
…t block

When a new block is imported in Tendermint consensus, CodeChain checks
the conditions below to judge whether it is safe to change the best
block:

First, whether the score is higher than before. Second, whether the
height of a new block is higher than the finalized block's
height.

This commit fixes the code that is calculating the finalized
block's height erroneously.
Immune users are immune from getting banned.
Before this commit, only the child of the best block and the child of
the best proposal block could be a new best proposal block. A block
that is a grandchild of the best block but not the child of the best
proposal block should be able to become a best proposal block.

After this commit, CodeChain checks whether a new block is the best
block's child or grandchild.
foriequal0 and others added 16 commits November 8, 2019 19:18
…1.17)

Update following packages:
bytes v0.4.7 -> v0.4.12
parking_lot_core v0.6.1 -> v0.6.2
toml v0.5.1 -> v0.5.3
tokio v0.1.7 -> v0.1.17

Commands were:
```
cargo update --package bytes
cargo update --package parking_lot_core:0.6.1
cargo update --package toml:0.5.1
cargo update --package tokio:0.1.7 --precise 0.1.17
```

Errors were:
1. bytes: jsonrpc requires ^0.4.8, and bytes were pinned to v0.4.7 for
sendgrid v0.8.1.
2. parking_lot_core: Amanieu/parking_lot#181
3. toml: `forward_to_deserialize_any_helper` was replaced by forward_to_deserialize_any
4. tokio: `Builder::core_threads` was introduced in 0.1.10
The fix (paritytech/jsonrpc#504) has been merged
to `master`. It'll take times to relase, so use the merged master directly.
When the data structure of data that is saved in the KVDB is changed,
CodeChain should migrate the data. To identify the stored data
structure, we need to keep a version number. This commit adds code to
save and load a version number. To make the DB easier to manage, this
code saves versions in the same column with the same prefix.
…lock

Tendermint worker's last_confirmed_view is used for two purposes. One
is the previous height's finalized view, and the other one is the
current height's finalized view. This commit is part of splitting the
last_confirmed_view variable.
Tendermint worker's last_confirmed_view is used for two purposes. One
is the previous height's finalized view, and the other one is the
current height's finalized view. This commit is part of splitting the
last_confirmed_view variable.
author_view: view when the block was created.
finalized_view: view when the block was finalized.
This patch
* prefers to use cmp
* makes code return early
* removes redundant cloning
* removes redundant lifetime annotation
* changes some other miscellaneous changes
to follow clippy update.
And rename some variables that was misnamed as tx_hash.
JSONRPC server 14.0.3 does not close HTTP server on connection error.
@sgkim126
Copy link
Contributor

sgkim126 commented Nov 8, 2019

I only reviewed the commits that I wrote. How about requesting reviews to @remagpie and @HoOngEe?

Copy link
Contributor

@remagpie remagpie left a comment

Choose a reason for hiding this comment

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

My commit is fine to be included.

Copy link
Contributor

@majecty majecty left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@HoOngEe HoOngEe left a comment

Choose a reason for hiding this comment

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

LGTM

@foriequal0 foriequal0 merged commit 6419957 into CodeChain-io:rc-2.2.0 Nov 11, 2019
@foriequal0 foriequal0 deleted the rc-2.2.0 branch November 11, 2019 02:35
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