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

Large keys are not stored in cache #2557

Closed
chejinge opened this issue Mar 28, 2024 · 3 comments · Fixed by #2849
Closed

Large keys are not stored in cache #2557

chejinge opened this issue Mar 28, 2024 · 3 comments · Fixed by #2849
Labels
✏️ Feature New feature or request

Comments

@chejinge
Copy link
Collaborator

chejinge commented Mar 28, 2024 •
edited
Loading

Which PikiwiDB functionalities are relevant/related to the feature request?

other

Description

cache中不存入大key

Proposed solution

cache中不存入大key
key维度string超过多少不存cache:16K
feild维度：cache-field-num-per-key 代码中已经实现，但是代码写的名字有问题

Alternatives considered

zset-cache-field-num-per-key改为cache-field-num-per-key
@chejinge chejinge added the ✏️ Feature New feature or request label Mar 28, 2024
@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.

Title: The big key is not stored in the cache

@chejinge chejinge changed the title cache中不存入大key Large keys are not stored in cache Mar 28, 2024
@AlexStocks
Copy link
Contributor

@changshunyu

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.

@长顺与

QlQlqiqi added a commit to QlQlqiqi/pika that referenced this issue Aug 7, 2024
@QlQlqiqi
feat: dont store cmd(Get/Set/Hget/Hset) with too large key in cache(fix…
151cab2 
…: OpenAtomFoundation#2557)
QlQlqiqi added a commit to QlQlqiqi/pika that referenced this issue Aug 7, 2024
@QlQlqiqi
feat: dont store cmd(Get/Set/Hget/Hset) with too large key in cache(fix…
40ca116 
…: OpenAtomFoundation#2557)
@QlQlqiqi QlQlqiqi mentioned this issue Aug 7, 2024
Merged
QlQlqiqi added a commit to QlQlqiqi/pika that referenced this issue Aug 7, 2024
@QlQlqiqi
feat: dont store cmd(Get/Set/Hget/Hset) with too large key in cache(fix…
427c9ef 
…: OpenAtomFoundation#2557)
QlQlqiqi added a commit to QlQlqiqi/pika that referenced this issue Aug 7, 2024
@QlQlqiqi
feat: dont store cmd(Get/Set/Hget/Hset) with too large key in cache(fix…
ba5903b 
…: OpenAtomFoundation#2557)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
✏️ Feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: dont store cmd(Get/Set/Hget/Hset) with too large key in cache QlQlqiqi/pika
3 participants
@AlexStocks @chejinge @Issues-translate-bot