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

fix:dbsize caculate error #2494

Merged
merged 4 commits into from
Mar 13, 2024
Merged

Conversation

chejinge
Copy link
Collaborator

@chejinge chejinge commented Mar 12, 2024

#2493

这里为什么不能做dbsize scard优化
首先:scard只是获取slotkey的数量,不是scan出所有的key的大小,scard slotkey的速度风险是可控的
其次:不能做一个数据保存下来,原因是假如在slotsreload,那前一秒和后一秒的keysize相差数据量太大,会不标准

@github-actions github-actions bot added the ☢️ Bug Something isn't working label Mar 12, 2024
src/pika_admin.cc Outdated Show resolved Hide resolved
@AlexStocks AlexStocks merged commit ef644aa into OpenAtomFoundation:unstable Mar 13, 2024
14 checks passed
luky116 pushed a commit to luky116/pika that referenced this pull request Mar 14, 2024
luky116 pushed a commit to luky116/pika that referenced this pull request Mar 14, 2024
luky116 pushed a commit that referenced this pull request Mar 18, 2024
* fix: dbsize caculate error
@chejinge chejinge deleted the dbsize branch March 31, 2024 01:30
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this pull request Jun 8, 2024
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.5.3 ☢️ Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants