-
Notifications
You must be signed in to change notification settings - Fork 38
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
Make metabase.ContainerSize
actually read-only
#1975
Conversation
From the `Bucket.ForEach` doc: ``` The provided function must not modify the bucket; this will result in undefined behavior. ``` Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
Codecov Report
@@ Coverage Diff @@
## master #1975 +/- ##
==========================================
- Coverage 30.50% 30.50% -0.01%
==========================================
Files 379 379
Lines 27705 27714 +9
==========================================
+ Hits 8452 8453 +1
- Misses 18557 18569 +12
+ Partials 696 692 -4
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
if err != nil { | ||
return fmt.Errorf("create global bucket for locked objects: %w", err) | ||
} | ||
bucketLocked := tx.Bucket(bucketNameLocked) |
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.
Maybe throw clear panic in this case instead of NPE?
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.
It is like this in many places and is not expected to happen.
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.
Changelog?
Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
Close #1944.
Close #1974.