update columnar bench (#2438) #3451
PSeitzAnnotations
1 error and 5 warnings
|
coverage
Process completed with exit code 1.
|
|
transmute used without annotations:
src/schema/document/default_document.rs#L556
warning: transmute used without annotations
--> src/schema/document/default_document.rs:556:32
|
556 | unsafe { std::mem::transmute(num) }
| ^^^^^^^^^ help: consider adding missing annotations: `transmute::<u8, schema::document::default_document::ValueType>`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#missing_transmute_annotations
= note: `#[warn(clippy::missing_transmute_annotations)]` on by default
|
|
useless use of `format!`:
src/indexer/mod.rs#L294
warning: useless use of `format!`
--> src/indexer/mod.rs:294:14
|
294 | (format!("{field_name_out_internal}"), Type::Str),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `field_name_out_internal.to_string()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format
= note: `#[warn(clippy::useless_format)]` on by default
|
|
redundant closure:
src/indexer/index_writer.rs#L2215
warning: redundant closure
--> src/indexer/index_writer.rs:2215:41
|
2215 | let ops: Vec<_> = (0..1000).map(|id| IndexingOp::add(id)).collect();
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `IndexingOp::add`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
= note: `#[warn(clippy::redundant_closure)]` on by default
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/indexer/index_writer.rs#L1580
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/indexer/index_writer.rs:1580:41
|
1580 | existing_ids.remove(&id);
| ^^^ help: change this to: `id`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/indexer/index_writer.rs#L1576
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/indexer/index_writer.rs:1576:41
|
1576 | existing_ids.remove(&id);
| ^^^ help: change this to: `id`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
= note: `#[warn(clippy::needless_borrow)]` on by default
|