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

Replace to use RocksDB for Tx Nonce and Canonical Chain ID #803

Merged
merged 8 commits into from
Feb 28, 2020

Conversation

earlbread
Copy link
Contributor

This replaces to use RocksDB for Tx Nonce and Canonical Chain ID in Libplanet.RocksDBStore. The remainder of using LiteDB will be replaced in the next PRs.

@earlbread earlbread self-assigned this Feb 28, 2020
@codecov
Copy link

codecov bot commented Feb 28, 2020

Codecov Report

Merging #803 into master will increase coverage by 0.06%.
The diff coverage is 97.05%.

@@            Coverage Diff             @@
##           master     #803      +/-   ##
==========================================
+ Coverage   86.92%   86.98%   +0.06%     
==========================================
  Files         237      239       +2     
  Lines       21245    21319      +74     
==========================================
+ Hits        18467    18545      +78     
- Misses       1456     1457       +1     
+ Partials     1322     1317       -5
Impacted Files Coverage Δ
Libplanet/Net/NetMQSocketExtensions.cs 81.63% <ø> (ø)
Libplanet.Tests/AddressExtensionsTest.cs 100% <ø> (ø)
...lanet/Serialization/SerializationInfoExtensions.cs 30% <ø> (ø)
Libplanet/Net/NetMQFrameExtensions.cs 100% <ø> (ø)
Libplanet/Store/StoreExtensions.cs 47.16% <ø> (ø)
Libplanet/Action/RandomExtensions.cs 100% <ø> (ø)
Libplanet/AddressExtensions.cs 100% <ø> (ø)
Libplanet.Stun/Stun/BytesConvertExtensions.cs 89.47% <ø> (ø)
...Tests/Stun/Attributes/StunAddressExtensionsTest.cs 100% <ø> (ø)
Libplanet/Address.cs 90.74% <ø> (ø) ⬆️
... and 15 more

CHANGES.md Outdated Show resolved Hide resolved
Libplanet/ByteArrayExtension.cs Outdated Show resolved Hide resolved
Libplanet/ByteArrayExtension.cs Outdated Show resolved Hide resolved
longfin
longfin previously approved these changes Feb 28, 2020
longfin
longfin previously approved these changes Feb 28, 2020
CHANGES.md Outdated
Comment on lines 13 to 14
- Extension classes have been renamed. However, it is not affected
if you do not call it directly. [[#803]]
Copy link
Contributor

@dahlia dahlia Feb 28, 2020

Choose a reason for hiding this comment

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

Suggested change
- Extension classes have been renamed. However, it is not affected
if you do not call it directly. [[#803]]
- Extension classes was renamed. However, it would not be affected
if you have called it by using instance method syntax. [[#803]]

Copy link
Contributor

Choose a reason for hiding this comment

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

This seems reverted?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My mistake. Restored it.

@earlbread
Copy link
Contributor Author

/rebase

Co-Authored-By: Hong Minhee <hong.minhee@gmail.com>
@dahlia dahlia requested a review from longfin February 28, 2020 12:58
@dahlia dahlia merged commit 8072c7e into planetarium:master Feb 28, 2020
@earlbread earlbread deleted the replace-litedb-1 branch February 28, 2020 13:38
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.

3 participants