Skip to content

Conversation

@hidedim
Copy link

@hidedim hidedim commented Oct 30, 2025

support 5 Operation.
TOPK.RESERVE key args(k, width, depth, decay): create TopK data structure.
TOPK.INFO key: reply information of args(k, width, depth, decay).
TOPK.ADD key value: add values to topk.
TOPK.QUERY key value: reply is value existed in topk.
TOPK.LIST key: reply item of topk.

Copy link
Member

@PragmaTwice PragmaTwice left a comment

Choose a reason for hiding this comment

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

Hi, thank you for your contribution!

Before we move on to the PR review, could you please provide a more detailed explanation of your design? For example, for features like timeseries (#3044) and t-digest (#2542), there were corresponding proposals that helped the community better understand the data structure design and the reasoning behind it. This makes it easier for community members to review the work and reach a consensus.

@hidedim
Copy link
Author

hidedim commented Oct 31, 2025

Hi, thank you for your contribution!

Before we move on to the PR review, could you please provide a more detailed explanation of your design? For example, for features like timeseries (#3044) and t-digest (#2542), there were corresponding proposals that helped the community better understand the data structure design and the reasoning behind it. This makes it easier for community members to review the work and reach a consensus.

Hi, thank you for the reminder and for this opportunity.
I will promptly prepare a detailed design document.

@aleksraiden
Copy link
Contributor

@hidedim lot of thanks about your work. Could you check issue with conflict?

@hidedim
Copy link
Author

hidedim commented Nov 17, 2025

@hidedim lot of thanks about your work. Could you check issue with conflict?
Thanks for your reply. I will complete the conflict check this week.

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