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

🎉[ᚬmaster] Rc/v0.10.0 #525

Merged
merged 180 commits into from
May 6, 2019
Merged

🎉[ᚬmaster] Rc/v0.10.0 #525

merged 180 commits into from
May 6, 2019

Conversation

doitian
Copy link
Member

@doitian doitian commented Apr 22, 2019

Bug Fixes

Features

Improvements

BREAKING CHANGES

  • This release has changed the underlying database format, please delete the old data directory before running the new version.
  • RPC get_live_cell returns (null, "unknown") when looking up null out point before, while returns (null, "live") now.
  • RPC uses string to encode all 64bit integrers now.
  • The executble ckb requires config files now, use ckb init to export the default ones.
  • The new features tx valid sicne (feat: tx valid since #372) and removal of version from Script (feat: Remove version from Script #422) have changed the core data structure. They affect both RPC and the P2P messages flatbuffers schema.

xxuejie and others added 30 commits April 8, 2019 01:20
412: chore: add make task to run integration test r=quake a=doitian



Co-authored-by: ian <ian@nervos.org>
414: fix: clear tx verfy cache when chain reorg r=doitian a=zhangsoledad



416: chore: ensure cfbc version r=zhangsoledad a=doitian



Co-authored-by: zhangsoledad <787953403@qq.com>
Co-authored-by: ian <ian@nervos.org>
419: chore: use new constructors to make fixed structs more readable r=zhangsoledad a=yangby-cryptape

Should I delete this unit test?

https://github.com/nervosnetwork/ckb/blob/32be9aa704fce9c269b93b247b64e5a87dece575/util/crypto/src/secp/signature.rs#L159-L173

Co-authored-by: Boyu Yang <yangby@cryptape.com>
NOTE: This is a breaking change
418: fix: try to repair a corrupted rocksdb automatically r=quake a=yangby-cryptape

I have test this patch with a corrupted database (from @TheWaWaR).

But I can not reproduce the bug by myself, so I did not write any test case.

The database works, but the data is still corrupted (**because `header` add new fields?**):

```
2019-04-09 07:39:24.154 +00:00 main WARN ckb_db::diskdb  Try repairing the rocksdb since Corruption: SST file is ahead of WALs ...
2019-04-09 07:39:24.384 +00:00 main WARN ckb_db::diskdb  Try opening the repaired rocksdb ...
thread 'main' panicked at 'header deserializing should be ok: Custom("invalid length 3, expected a 0x-prefixed hex string with 64 digits")', src/libcore/result.rs:997:5
stack backtrace:
```

Ref:
- [RocksDB: Repairer](https://github.com/facebook/rocksdb/wiki/RocksDB-Repairer)
- [RocksDB: WAL Recovery Modes](https://github.com/facebook/rocksdb/wiki/WAL-Recovery-Modes)
- [`pub fn set_wal_recovery_mode(&mut self, mode: DBRecoveryMode)`](https://github.com/rust-rocksdb/rust-rocksdb/blob/267d92cbf9fdcbce9052ef06d7cb90b3111c6c54/src/db_options.rs#L1042-L1046)

Co-authored-by: Boyu Yang <yangby@cryptape.com>
422: feat: Remove version from Script r=doitian a=xxuejie

NOTE: This is a breaking change

Co-authored-by: Xuejie Xiao <xxuejie@gmail.com>
404: feat: Update embedded script r=doitian a=xxuejie

The embedded secp256k1_blake2b_sighash_all script is compiled from https://github.com/nervosnetwork/ckb-system-scripts/blob/384da8adc0564a228f54ae4122adda12d966f9bc/c/secp256k1_blake2b_sighash_all.c

Co-authored-by: Xuejie Xiao <xxuejie@gmail.com>
Bumps [flatbuffers-verifier](https://github.com/nervosnetwork/cfb) from 0.1.8 to 0.1.9.
- [Release notes](https://github.com/nervosnetwork/cfb/releases)
- [Commits](https://github.com/nervosnetwork/cfb/commits)

Signed-off-by: dependabot[bot] <support@dependabot.com>
Bumps [bincode](https://github.com/TyOverby/bincode) from 1.1.2 to 1.1.3.
- [Release notes](https://github.com/TyOverby/bincode/releases)
- [Commits](bincode-org/bincode@v1.1.2...v1.1.3)

Signed-off-by: dependabot[bot] <support@dependabot.com>
Bumps [serde_bytes](https://github.com/serde-rs/bytes) from 0.10.5 to 0.11.1.
- [Release notes](https://github.com/serde-rs/bytes/releases)
- [Commits](serde-rs/bytes@0.10.5...0.11.1)

Signed-off-by: dependabot[bot] <support@dependabot.com>
Bumps [serde](https://github.com/serde-rs/serde) from 1.0.89 to 1.0.90.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.89...v1.0.90)

Signed-off-by: dependabot[bot] <support@dependabot.com>
xxuejie and others added 28 commits April 19, 2019 04:44
hotfix: fix VM hang bug for certain invalid programs
fix: several patches from testnet

BREAKING CHANGE:

- RPC `get_live_cell` returns `(null, "unknown")` when looking up null out point before, while returns `(null, "live")` now.

---
commit 5e51d65
Author: zhangsoledad <787953403@qq.com>
Date:   Wed Apr 17 17:23:14 2019 +0800

    fix: proposal table

commit 2983480
Author: Linfeng Qian <thewawar@gmail.com>
Date:   Sat Apr 13 22:31:18 2019 +0800

    revert: Insert repeat transaction in pool

commit e74668b
Author: zhangsoledad <787953403@qq.com>
Date:   Sat Apr 13 20:46:48 2019 +0800

    fix: block download

commit db31c9d
Author: zhangsoledad <787953403@qq.com>
Date:   Sat Apr 13 18:39:00 2019 +0800

    fix: conflict transaction verify
feat: add parameters to control the block generation rate in dummy mode

When I do tests in my laptop, the speed of product block is too slow.

Ref: #389
…lease

chore: enable overflow-checks build flag on release mode
A wrong version is selected when merged into develop from v0.9.0.
…h_all

[ᚬrc/v0.10.0] fix: revert secp256k1_blake160_sighash_all
@doitian doitian merged commit c5f9d94 into master May 6, 2019
@doitian doitian deleted the rc/v0.10.0 branch May 6, 2019 12:40
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.

None yet

10 participants