8000 [opt]: succinct trie by stone1100 · Pull Request #990 · lindb/lindb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[opt]: succinct trie #990

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
Merged
merged 1 commit into from
Nov 10, 2023
Merged

[opt]: succinct trie #990

merged 1 commit into from
Nov 10, 2023

Conversation

stone1100
Copy link
Member
@stone1100 stone1100 commented Nov 10, 2023 •
edited
Loading

What problem does this PR solve?

Issue Number: #xxx

Problem Summary:

  • modify build trie logic, improve memory usage
  • go version
  • gomock

Check List

Tests

  • Unit test
  • Integration test
  • No code

@stone1100 stone1100 force-pushed the new_trie branch 2 times, most recently from 74d8677 to 0e0dc82 Compare November 10, 2023 02:07
@codecov Codecov
Copy link
codecov bot commented Nov 10, 2023 •
edited
Loading

Codecov Report

Merging #990 (d15cc82) into main (081f006) will decrease coverage by 0.29%.
The diff coverage is 80.00%.

❗ Current head d15cc82 differs from pull request most recent head fe49675. Consider uploading reports for the commit fe49675 to get more accurate results

@@            Coverage Diff             @@
##             main     #990      +/-   ##
==========================================
- Coverage   96.54%   96.25%   -0.29%     
==========================================
  Files         401      405       +4     
  Lines       24944    24907      -37     
==========================================
- Hits        24081    23974     -107     
- Misses        609      662      +53     
- Partials      254      271      +17
Files Coverage Δ
cmd/cli/main.go 89.71% <ø> (ø)
coordinator/broker/state_manager.go 100.00% <ø> (ø)
coordinator/discovery/registry.go 100.00% <ø> (ø)
coordinator/discovery/state_machine.go 100.00% <ø> (ø)
coordinator/elect/election.go 100.00% <ø> (ø)
coordinator/master/state_manager.go 100.00% <ø> (ø)
coordinator/root/state_manager.go 100.00% <ø> (ø)
coordinator/storage/state_manager.go 100.00% <ø> (ø)
flow/node_choose.go 100.00% <100.00%> (ø)
internal/conntrack/grpc_client_tracker.go 100.00% <ø> (ø)
... and 24 more

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@stone1100 stone1100 force-pushed the new_trie branch 2 times, most recently from 87e181a to 0b55fe7 Compare November 10, 2023 02:43
@stone1100 stone1100 merged commit 4d0cbac into lindb:main Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@stone1100
0