-
Notifications
You must be signed in to change notification settings - Fork 315
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.
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
feat: renaming kv directory to metadata #2549
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #2549 +/- ##
===========================================
- Coverage 85.34% 85.02% -0.33%
===========================================
Files 737 737
Lines 117409 117737 +328
===========================================
- Hits 100206 100101 -105
- Misses 17203 17636 +433 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@waynexia Please take a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @Lilit0x
* fix: renamed kv directory to metadata directory * fix: changed function name * fix: changed function name
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
The KV directory has been renamed to metadata to provide more useful info for users.
Checklist
#2493