Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Allow neo-cli to switch between RocksDB and LevelDB #451

Closed
wants to merge 3 commits into from

Conversation

shargon
Copy link
Member

@shargon shargon commented Aug 27, 2019

Require changes when neo-project/neo#1063 was merged

If you want to test RocksDB PR, you can use this branch

This was referenced Aug 27, 2019
@vncoelho
Copy link
Member

Looks simple, @shargon. Nice job.
In this sense, even after #1063 we are going to be able to use both of them?

@shargon
Copy link
Member Author

shargon commented Aug 27, 2019

Yes, we can do real benchmarks between them now

@@ -1,22 +1,31 @@

Copy link
Member Author

Choose a reason for hiding this comment

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

This changes will be removed on the last version

<PackageReference Include="System.ServiceProcess.ServiceController" Version="4.5.0" />
</ItemGroup>

<ItemGroup>
<ProjectReference Include="..\..\neo\neo\neo.csproj" />
</ItemGroup>
Copy link
Member Author

Choose a reason for hiding this comment

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

This changes will be removed on the last version

@shargon shargon requested a review from erikzhang August 27, 2019 13:35
@erikzhang erikzhang closed this Dec 1, 2019
@erikzhang erikzhang reopened this Dec 1, 2019
@erikzhang erikzhang closed this Dec 1, 2019
@shargon shargon deleted the rocks-db branch December 1, 2019 15:29
Jim8y pushed a commit to Jim8y/neo-node that referenced this pull request Aug 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants