Skip to content

Commit e42cf0e

Browse files
committed
fixes
1 parent ab2a384 commit e42cf0e

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

src/errors/schema_error.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ impl TableReferenceProto {
168168
impl SchemaErrorProto {
169169
pub fn from_schema_error(err: &SchemaError, backtrace: Option<&String>) -> Self {
170170
match err {
171-
SchemaError::AmbiguousReference { ref field } => SchemaErrorProto {
171+
SchemaError::AmbiguousReference { field } => SchemaErrorProto {
172172
inner: Some(SchemaErrorInnerProto::AmbiguousReference(
173173
AmbiguousReferenceProto {
174174
field: Some(ColumnProto::from_column(field)),

src/metrics/proto.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
#[cfg(feature = "integration")]
12
use chrono::DateTime;
23
use datafusion::common::internal_err;
34
use datafusion::error::DataFusionError;
@@ -381,10 +382,10 @@ pub fn metric_proto_to_df(metric: MetricProto) -> Result<Arc<Metric>, DataFusion
381382
)))
382383
}
383384
Some(MetricValueProto::StartTimestamp(start_ts)) => match start_ts.value {
384-
Some(value) => {
385+
Some(_value) => {
385386
let timestamp = Timestamp::new();
386387
#[cfg(feature = "integration")]
387-
timestamp.set(DateTime::from_timestamp_nanos(value));
388+
timestamp.set(DateTime::from_timestamp_nanos(_value));
388389
#[cfg(not(feature = "integration"))]
389390
{ /* DateTime not available without integration feature */ }
390391
Ok(Arc::new(Metric::new_with_labels(
@@ -400,10 +401,10 @@ pub fn metric_proto_to_df(metric: MetricProto) -> Result<Arc<Metric>, DataFusion
400401
))),
401402
},
402403
Some(MetricValueProto::EndTimestamp(end_ts)) => match end_ts.value {
403-
Some(value) => {
404+
Some(_value) => {
404405
let timestamp = Timestamp::new();
405406
#[cfg(feature = "integration")]
406-
timestamp.set(DateTime::from_timestamp_nanos(value));
407+
timestamp.set(DateTime::from_timestamp_nanos(_value));
407408
#[cfg(not(feature = "integration"))]
408409
{ /* DateTime not available without integration feature */ }
409410
Ok(Arc::new(Metric::new_with_labels(

src/protobuf/distributed_codec.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ use std::sync::Arc;
1919
pub struct DistributedCodec;
2020

2121
impl DistributedCodec {
22-
pub fn new_combined_with_user(cfg: &SessionConfig) -> impl PhysicalExtensionCodec {
22+
pub fn new_combined_with_user(cfg: &SessionConfig) -> impl PhysicalExtensionCodec + use<> {
2323
let mut combined_codec = ComposedPhysicalExtensionCodec::default();
2424
combined_codec.push(DistributedCodec {});
2525
if let Some(ref user_codec) = get_distributed_user_codec(cfg) {

0 commit comments

Comments
 (0)