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

refactor: use rocksdb_wrapper::write_batch_put_ctx to reimplement multi_put #666

Merged
merged 3 commits into from
Jan 4, 2021

Conversation

levy5307
Copy link
Contributor

@levy5307 levy5307 commented Dec 31, 2020

What problem does this PR solve?

use rocksdb_wrapper::write_batch_put_ctx to reimplement multi_put.

  • Manual test (add detailed scripts or steps below)
>>> use temp
OK
>>> ls -d
[general_info]
app_id  status     app_name  app_type  partition_count  replica_count  is_stateful  create_time          drop_time  drop_expire  envs_count  
1       AVAILABLE  stat      pegasus   4                3              true         2020-12-31_14:40:35  -          -            0           
2       AVAILABLE  temp      pegasus   8                3              true         2020-12-31_14:40:35  -          -            0           

[healthy_info]
app_id  app_name  partition_count  fully_healthy  unhealthy  write_unhealthy  read_unhealthy  
1       stat      4                4              0          0                0               
2       temp      8                8              0          0                0               

[summary]
total_app_count            : 2
fully_healthy_app_count    : 2
unhealthy_app_count        : 0
write_unhealthy_app_count  : 0
read_unhealthy_app_count   : 0

>>> multi_set hash sort1 value1 sort2 value2 sort3 value3
OK

app_id          : 2
partition_index : 4
decree          : 5
server          : 10.232.55.210:34803
>>> multi_get hash sort1 sort2 sort3
"hash" : "sort1" => "value1"
"hash" : "sort2" => "value2"
"hash" : "sort3" => "value3"

3 key-value pairs got, fetch completed.

app_id          : 2
partition_index : 4
server          : 10.232.55.210:34803
>>> 

@levy5307 levy5307 merged commit af0b3df into apache:master Jan 4, 2021
@neverchanje neverchanje mentioned this pull request Mar 1, 2021
@levy5307 levy5307 deleted the write-multi-put branch March 18, 2021 03:16
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.

4 participants