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

Fix several issues of topN pre-calculation #455

Merged
merged 9 commits into from
May 29, 2024
Merged

Fix several issues of topN pre-calculation #455

merged 9 commits into from
May 29, 2024

Conversation

hanahmily
Copy link
Contributor

@hanahmily
Fix the aggregation field name is not "value"
4977fe1 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Update debug info to decode series
d2b6ee5 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Never marshal AnyTag into written data
f881e23 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Encode int
083a621 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Add log to flow
d946346 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Limit flush interval
dd08b64 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Use server time to flush topn
b557614 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Fix lint issues
4f4bc4b 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@hanahmily
Update CHANGES.md
b978ab8 
Signed-off-by: Gao Hongtao <hanahmily@gmail.com>
@wu-sheng wu-sheng added this to the 0.7.0 milestone May 29, 2024
wu-sheng
wu-sheng approved these changes May 29, 2024
View reviewed changes
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

I renamed the milestone name to match your changelogs. And created new 0.7 milestone.

@wu-sheng wu-sheng added the bug Something isn't working label May 29, 2024
@wu-sheng wu-sheng merged commit 8906b3c into main May 29, 2024
14 checks passed
@wu-sheng wu-sheng deleted the topn-val branch May 29, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wu-sheng wu-sheng wu-sheng approved these changes

@lujiajing1126 lujiajing1126 Awaiting requested review from lujiajing1126

@ButterBright ButterBright Awaiting requested review from ButterBright

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
0.6.1
2 participants
@hanahmily @wu-sheng