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

Add tests for KvBackend trait implements #2618

Closed
Tracked by #2616
WenyXu opened this issue Oct 18, 2023 · 2 comments · Fixed by #3700
Closed
Tracked by #2616

Add tests for KvBackend trait implements #2618

WenyXu opened this issue Oct 18, 2023 · 2 comments · Fixed by #3700
Assignees
@poltao
Labels
C-enhancement Category Enhancements good first issue Good for newcomers help wanted Extra attention is needed

Comments

@WenyXu
Copy link
Member

WenyXu commented Oct 18, 2023 •
edited
Loading

What type of enhancement is this?

Tech debt reduction

What does the enhancement do?

Add behavior tests for KvBackend implements to ensure all implements have the same behaviors.
@WenyXu WenyXu mentioned this issue Oct 18, 2023
Closed
2 tasks
@WenyXu WenyXu added good first issue Good for newcomers C-enhancement Category Enhancements help wanted Extra attention is needed Difficulty: Mid labels Oct 18, 2023
@poltao
Copy link
Contributor

poltao commented Nov 14, 2023

@WenyXu Can I try it? It a good chance to learn about KvBackend.

@WenyXu
Copy link
Member Author

WenyXu commented Nov 14, 2023

@WenyXu Can I try it? It a good chance to learn about KvBackend.

Thks ❤️

@poltao poltao mentioned this issue Apr 12, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@poltao poltao

Labels
C-enhancement Category Enhancements good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: Add tests for KvBackend trait implement poltao/greptimedb
2 participants
@WenyXu @poltao