Skip to content

Commit

Permalink
fixup! fixup! fixup! Apply changes from code review
Browse files Browse the repository at this point in the history
Signed-off-by: Christian Haudum <christian.haudum@gmail.com>
  • Loading branch information
chaudum committed Sep 5, 2024
1 parent 04d3d14 commit ea8a541
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 14 deletions.
2 changes: 0 additions & 2 deletions pkg/storage/bloom/v1/bloom_builder.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,6 @@ func (b *BloomBlockBuilder) Append(bloom *Bloom) (BloomOffset, error) {
}
}

// version := b.opts.Schema.version

b.scratch.Reset()
if err := bloom.Encode(b.scratch); err != nil {
return BloomOffset{}, errors.Wrap(err, "encoding bloom")
Expand Down
12 changes: 0 additions & 12 deletions pkg/storage/bloom/v1/builder.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,18 +66,6 @@ func (b BlockOptions) Encode(enc *encoding.Encbuf) {
enc.PutBE64(b.BlockSize)
}

// func NewDefaultBlockOptions(maxBlockSizeBytes, maxBloomSizeBytes uint64) BlockOptions {
// opts := NewBlockOptionsFromSchema(Schema{
// version: DefaultSchemaVersion,
// encoding: chunkenc.EncNone,
// nGramLength: 0,
// nGramSkip: 0,
// })
// opts.BlockSize = maxBlockSizeBytes
// opts.UnencodedBlockOptions.MaxBloomSizeBytes = maxBloomSizeBytes
// return opts
// }

func NewBlockOptions(enc chunkenc.Encoding, nGramLength, nGramSkip, maxBlockSizeBytes, maxBloomSizeBytes uint64) BlockOptions {
opts := NewBlockOptionsFromSchema(Schema{
version: CurrentSchemaVersion,
Expand Down

0 comments on commit ea8a541

Please sign in to comment.