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

Remove stats resolver #266

Closed
5 tasks done
0xERR0R opened this issue Sep 6, 2021 · 0 comments · Fixed by #279
Closed
5 tasks done

Remove stats resolver #266

0xERR0R opened this issue Sep 6, 2021 · 0 comments · Fixed by #279
Milestone
v0.16

Comments

@0xERR0R
Copy link
Owner

0xERR0R commented Sep 6, 2021 •
edited
Loading

Remove stats resolver with statistic aggregation. database query logging (#258) with new Grafana dashboard (#265) should be used instead. This would allow to save some memory, since the aggregation is in-memory only.

Following should be removed:

  • statsResolver
  • stats package
  • Signal handler to show the stats on the console
  • dependency to go-pretty (tables for CLI)
  • documentation
@0xERR0R 0xERR0R added this to the 0.16 milestone Sep 6, 2021
0xERR0R added a commit that referenced this issue Sep 13, 2021
@0xERR0R
Remove stats resolver (#266)
bf6a0d7
@0xERR0R 0xERR0R mentioned this issue Sep 13, 2021
Merged
0xERR0R added a commit that referenced this issue Sep 13, 2021
@0xERR0R
Remove stats resolver (#266)
43bd568
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.16
Development

Successfully merging a pull request may close this issue.

Remove stats resolver (#266) 0xERR0R/blocky
1 participant
@0xERR0R