-
Notifications
You must be signed in to change notification settings - Fork 314
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
chore: log error for detail #4011
chore: log error for detail #4011
Conversation
Reference to our style guide: https://github.com/GreptimeTeam/greptimedb/blob/main/docs/style-guide.md#error-handling |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4011 +/- ##
==========================================
- Coverage 85.48% 85.16% -0.33%
==========================================
Files 977 977
Lines 169462 169519 +57
==========================================
- Hits 144859 144365 -494
- Misses 24603 25154 +551 |
2f19846
to
54164f1
Compare
54164f1
to
d80d6d3
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
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
As the title said.
Checklist