scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-10-08 11:57:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

add nolint directive

Browse source 
there isn't much piont in making snapshot take a WriterTo, because the only
thing that's going to be written to a fragment is a roaring bitmap. making it
WriterTo just makes it slightly more annoying to jump into the Bitmap.WriterTo
implementation. since it's a private function, it should be straightforward to
change this in the future if the need arises.
This commit is contained in:
Matt Jaffee 2018-09-12 17:31:57 -05:00
parent 4a5f12631c
commit 79c783eaf8
No known key found for this signature in database
GPG key ID: 08A3DFFF987B11BF
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
fragment.go
View file

@ -1496,7 +1496,7 @@ func (f *fragment) snapshot() error {
// snapshot writes the fragment f with bm as the data. It is unprotected, and
// f.mu must be locked when calling it.
func snapshot(f *fragment, bm *roaring.Bitmap) error {
func snapshot(f *fragment, bm *roaring.Bitmap) error { // nolint: interfacer
f.Logger.Printf("fragment: snapshotting %s/%s/%s/%d", f.index, f.field, f.view, f.shard)
completeMessage := fmt.Sprintf("fragment: snapshot complete %s/%s/%s/%d", f.index, f.field, f.view, f.shard)