Skip to content

Commit

Permalink
fix 180 clippy
Browse files Browse the repository at this point in the history
  • Loading branch information
xinlifoobar committed Aug 1, 2024
1 parent 0faaa26 commit d082d51
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -517,7 +517,6 @@ macro_rules! make_data_page_stats_iterator {
}
}

#[allow(clippy::redundant_closure_call)]
impl<'a, I> Iterator for $iterator_type<'a, I>
where
I: Iterator<Item = (usize, &'a Index)>,
Expand Down
2 changes: 1 addition & 1 deletion datafusion/functions-aggregate/src/approx_distinct.rs
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,7 @@ impl AggregateUDFImpl for ApproxDistinct {
other => {
return not_impl_err!(
"Support for 'approx_distinct' for data type {other} is not implemented"
)
)
}
};
Ok(accumulator)
Expand Down
1 change: 1 addition & 0 deletions datafusion/substrait/src/serializer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ use substrait::proto::Plan;
use std::fs::OpenOptions;
use std::io::{Read, Write};

#[allow(clippy::suspicious_open_options)]
pub async fn serialize(sql: &str, ctx: &SessionContext, path: &str) -> Result<()> {
let protobuf_out = serialize_bytes(sql, ctx).await;
let mut file = OpenOptions::new().create(true).write(true).open(path)?;
Expand Down

0 comments on commit d082d51

Please sign in to comment.