-
Notifications
You must be signed in to change notification settings - Fork 316
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
refactor: allocate table ids in the procedure #3293
refactor: allocate table ids in the procedure #3293
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3293 +/- ##
==========================================
- Coverage 85.42% 84.99% -0.43%
==========================================
Files 889 889
Lines 145477 145471 -6
==========================================
- Hits 124272 123646 -626
- Misses 21205 21825 +620 |
d3b03e5
to
0c9cb66
Compare
I'm going to drop the duplicate commits after #3271 is merged |
aaf5ab1
to
43f12cc
Compare
7b793cc
to
227cf24
Compare
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 PTAL |
31d0b57
to
627a86d
Compare
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Checklist
Refer to a related PR or issue link (optional)
#3233
Waits for #3271