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.

Sign up for GitHub

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

Jump to bottom

Use rocksdb as default store #80

Merged
merged 36 commits into from
Mar 14, 2019
Merged

Use rocksdb as default store #80

merged 36 commits into from
Mar 14, 2019

Conversation

gdiazlo
Copy link
Contributor

@gdiazlo gdiazlo commented Mar 13, 2019

No description provided.

@gdiazlo gdiazlo requested a review from aalda March 13, 2019 21:25
aalda and others added 29 commits March 14, 2019 11:57
@aalda @gdiazlo
Add rocksdb submodule
0e02418
@aalda @gdiazlo
Set rocksdb submodule to release v5.18.3
f083b37
@gdiazlo
Use http instead of ssh in git submodule url
e2496e0
@aalda @gdiazlo
Add snappy c dependency
9a6e508
@aalda @gdiazlo
Add base skeleton for rocksdb bindings
5af4c89
@gdiazlo
Add procedure to use static libraries when building rocksdb bindings
692b20d
@gdiazlo
Remove snappy from git modules
6e23cdb
@gdiazlo
build rocksdb using static_lib target
f230aa9
@gdiazlo
Add c-deps as submodules
1abf0b6
@gdiazlo
wip: rocksdb binding ld options
e22dbf2
@gdiazlo
Fix typo on c-deps library path for LDFLAGS
cb933b1
@gdiazlo
Fix buildeps
e7838e2
@gdiazlo
clean up build deps
05bfa8f
@aalda @gdiazlo
Implement rocksdb basic storage
97b0216
@gdiazlo
Remove unneeded dependencies
505004b
@aalda @gdiazlo
Fix rocksdb flags to work on darwin
b627664
@aalda @gdiazlo
Implement backup and load methods
dc377ff
@gdiazlo
Use rocksdb when testing api
3a4d0ee
@gdiazlo
Use rocksdb when testing
08ffccf
@gdiazlo
Make getAlerts to unmarshal json, and skip failing tests waiting for …
c1dd7fe 
…agents work
@gdiazlo
Implement snapshot storage api in rocksdb and badger, and update raft…
7fa84a2 
…wall to use it for backup and restore
@gdiazlo
Add protobuf to dependencies
e26eda1
@gdiazlo
Use rocksdb as default storage engine
8f31c44
@gdiazlo
update azure pipeline
f4f7fdf
@gdiazlo
Update README with new building instructions
522c3ed
@gdiazlo
Enable PR building on all branches by default
12c161f
@gdiazlo
Revert "Update README with new building instructions"
a47b531 
This reverts commit 7efdb22.
@gdiazlo
Fix azure pipeline syntax error in yaml
a1dce44
@gdiazlo
add working directory to rocksdb construction step
e303a6b
aalda
aalda reviewed Mar 14, 2019
View reviewed changes
Copy link
Contributor

@aalda aalda left a comment

Choose a reason for hiding this comment

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

LGTM

@aalda aalda merged commit 2951295 into BBVA:master Mar 14, 2019
@aalda aalda deleted the rocksdb branch March 25, 2019 14:30
suizman pushed a commit to jllucas/qed that referenced this pull request Jun 4, 2019
@aalda
Merge pull request BBVA#80 from aalda/rocksdb
1657ad4 
Use rocksdb as default store

Former-commit-id: 2951295
suizman pushed a commit that referenced this pull request Sep 30, 2019
@aalda
Merge pull request #80 from aalda/rocksdb
1e9d866 
Use rocksdb as default store

Former-commit-id: 00e30bb [formerly 2951295]
Former-commit-id: 1657ad4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aalda aalda aalda left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gdiazlo @aalda