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

[Compatibility] Added EXPIREAT and PEXPIREAT command and bug fixes for EXPIRE and PEXPIRE #666

Merged
merged 10 commits into from
Oct 4, 2024

Conversation

Vijay-Nirmal
Copy link
Contributor

Adding EXPIREAT and PEXPIREAT command to garnet

  • Add EXPIREAT and PEXPIREAT command
  • Add Test cases
  • Add documentation
  • Fix a bug in EXPIRE/PEXPIRE where it doesn't support valid option combination
  • Fix a bug in EXPIRE/PEXPIRE where it doesn't treat non-volatile key as an infinite TTL for GT and LT option

@TalZaccai TalZaccai self-requested a review September 18, 2024 18:12
Copy link
Contributor

@TalZaccai TalZaccai left a comment

Choose a reason for hiding this comment

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

Great additions! Thanks for adding the options to EXPIRE / PEXPIRE! Added a couple of comments.

libs/server/ExpireOption.cs Show resolved Hide resolved
libs/server/Resp/KeyAdminCommands.cs Show resolved Hide resolved
libs/server/Storage/Session/MainStore/MainStoreOps.cs Outdated Show resolved Hide resolved
@Vijay-Nirmal
Copy link
Contributor Author

@TalZaccai Fixed the review comments except one outstand question in the review comment

libs/server/Resp/KeyAdminCommands.cs Outdated Show resolved Hide resolved
libs/server/Resp/Parser/RespCommand.cs Outdated Show resolved Hide resolved
@Vijay-Nirmal
Copy link
Contributor Author

@TalZaccai Fixed all review comments

Copy link
Contributor

@TalZaccai TalZaccai left a comment

Choose a reason for hiding this comment

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

small nit :)

Copy link
Contributor

@TalZaccai TalZaccai left a comment

Choose a reason for hiding this comment

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

Great work, thank you!

@TalZaccai TalZaccai merged commit 748ec9a into microsoft:main Oct 4, 2024
15 checks passed
@Vijay-Nirmal Vijay-Nirmal deleted the new/expireat-and-pexpireat branch October 4, 2024 19:27
@TedHartMS
Copy link
Contributor

TedHartMS commented Oct 4, 2024 via email

@Vijay-Nirmal
Copy link
Contributor Author

@TedHartMS

EvilLaughTheDevilGIF

@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants