Influxdata
/
influxdb
mirror of https://github.com/influxdata/influxdb.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix: Remove let bindings to the unit value found by clippy 

The union and intersect methods on RowIDs don't return any value, so
these let bindings were always getting the unit value. Then the if/else
block that the bindings were returned from wasn't being assigned to
anything, so removing the binding and the return of the unit value from
these blocks is fine.
pull/24376/head
Carol (Nichols || Goulding) 2022-07-01 11:36:24 -04:00
parent 03aa338897
commit 69a3ad95b6
No known key found for this signature in database
GPG Key ID: E907EE5A736F87D4
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
read_buffer/src/row_group.rs
View File

@ -441,13 +441,11 @@ impl RowGroup {
RowIDsOption::Some(row_ids) => {
let now = std::time::Instant::now();
if result_row_ids.is_empty() {
let row_ids = result_row_ids.union(&row_ids);
result_row_ids.union(&row_ids);
trace!(elapsed=?now.elapsed(), "unioning row IDs");
row_ids
} else {
let row_ids = result_row_ids.intersect(&row_ids);
result_row_ids.intersect(&row_ids);
trace!(elapsed=?now.elapsed(), "intersecting row IDs");
row_ids
}
// before evaluating the next expression check if we have