pika icon indicating copy to clipboard operation
pika copied to clipboard

Published 20 hours ago • verified publisher icon OpenAtomFoundation

rocksdb: support seek perfomence

Open wanghenshui opened this issue 2 years ago • 7 comments

wanghenshui avatar Feb 21 '23 11:02 wanghenshui

https://github.com/Qihoo360/blackwidow/pull/17

wanghenshui avatar Feb 21 '23 16:02 wanghenshui

是不是把我之前的 patch 搬过来，然后补充下 benchmark

WyattJia avatar Mar 07 '23 06:03 WyattJia

是不是把我之前的 patch 搬过来，然后补充下 benchmark

approve

kernelai avatar Mar 07 '23 06:03 kernelai

是不是把我之前的 patch 搬过来，然后补充下 benchmark

approve

assgin to me

WyattJia avatar Mar 07 '23 07:03 WyattJia

相应 PR https://github.com/Qihoo360/blackwidow/pull/17 在 seek 的时候指定 iterator range 上下边界，但有问题：

image

里面 list RedisLists::LRem 函数这个地方，read_options 这里他设置值了，但是 "rocksdb::Iterator* iter = db_->NewIterator(default_read_options_, handles_[1]); " 这里使用的还是 default_read_options_

另外，这个 PR 已经 2022 年合并了，但是 Pika 仓库 src/storage 里面 Blackwidow 最新的代码里没这个 PR 的代码，很神奇

AlexStocks avatar Feb 08 '24 09:02 AlexStocks

reassgin to me

WyattJia avatar Feb 13 '24 06:02 WyattJia

reassgin to me

thank you.

AlexStocks avatar Feb 18 '24 08:02 AlexStocks