Skip to content

Switch from fork to orginal#24

Merged
tac0turtle merged 2 commits intomasterfrom
marko/switch_rocksdb
Oct 18, 2019
Merged

Switch from fork to orginal#24
tac0turtle merged 2 commits intomasterfrom
marko/switch_rocksdb

Conversation

@tac0turtle
Copy link
Contributor

@tac0turtle tac0turtle commented Oct 17, 2019

Signed-off-by: Marko Baricevic marbar3778@yahoo.com

Fixes #18

- gorocksdb merged what we needed: tecbot/gorocksdb#174

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
Copy link
Contributor

@Stumble Stumble 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

@melekes melekes left a comment

Choose a reason for hiding this comment

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

tm-db/db.go

Line 35 in 92fd030

// RocksDBBackend represents rocksdb (uses Stumble fork github.com/stumble/gorocksdb of the tecbot wrapper)
needs to be updated as well. thanks!

@tac0turtle tac0turtle merged commit b6ac720 into master Oct 18, 2019
@tac0turtle tac0turtle deleted the marko/switch_rocksdb branch October 18, 2019 12:50
faddat pushed a commit to notional-labs/tm-db that referenced this pull request May 8, 2022
tac0turtle added a commit that referenced this pull request Aug 25, 2022
Co-authored-by: Marko <marbar3778@yahoo.com>
javiersuweijie pushed a commit to terra-money/tm-db that referenced this pull request Mar 16, 2023
* Update Go version to 1.18

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Update readme to reflect minimum Go version

Signed-off-by: Thane Thomson <connect@thanethomson.com>

Signed-off-by: Thane Thomson <connect@thanethomson.com>
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.

Switch to original techbot rocksdb wrapper

3 participants