Skip to content

Commit d5cf16d

Browse files
committed
fix clippy errors
1 parent e2f1e1c commit d5cf16d

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

src/context.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -767,7 +767,7 @@ impl PySessionContext {
767767
// Extract schema data if available to avoid borrowing
768768
let schema_owned = schema.map(|s| s.0.clone());
769769

770-
let _ = if path.is_instance_of::<PyList>() {
770+
if path.is_instance_of::<PyList>() {
771771
let paths = path.extract::<Vec<String>>()?;
772772
// Clone self to avoid borrowing
773773
let self_clone = self.clone();

src/substrait.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ impl PySubstraitSerializer {
116116
// Create a future that moves owned values
117117
let future = async move { serializer::serialize_bytes(&sql_owned, &ctx_owned).await };
118118

119-
let proto_bytes: Vec<u8> = wait_for_future(py, future)??.into();
119+
let proto_bytes: Vec<u8> = wait_for_future(py, future)??;
120120
Ok(PyBytes::new(py, &proto_bytes).into())
121121
}
122122

src/utils.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ where
8585
}
8686
_ = interval.tick() => {
8787
// Time to check Python signals
88+
#[allow(clippy::question_mark)]
8889
if let Err(py_exc) = Python::with_gil(|py| py.check_signals()) {
8990
return Err(py_exc);
9091
}

0 commit comments

Comments
 (0)