This is an automated email from the ASF dual-hosted git repository.
wayne pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new 9c065065df fix: CI compilation failed on substrait (#10683)
9c065065df is described below
commit 9c065065dfb4b87bf3f32c0c83feed55a44812df
Author: Jonah Gao <[email protected]>
AuthorDate: Mon May 27 11:44:51 2024 +0800
fix: CI compilation failed on substrait (#10683)
---
datafusion/substrait/src/logical_plan/producer.rs | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/datafusion/substrait/src/logical_plan/producer.rs
b/datafusion/substrait/src/logical_plan/producer.rs
index 400609ff14..592b40db59 100644
--- a/datafusion/substrait/src/logical_plan/producer.rs
+++ b/datafusion/substrait/src/logical_plan/producer.rs
@@ -1411,7 +1411,7 @@ fn to_substrait_type(dt: &DataType, nullable: bool) ->
Result<substrait::proto::
parameter:
Some(parameter::Parameter::DataType(substrait::proto::Type {
kind: Some(r#type::Kind::I32(r#type::I32 {
type_variation_reference: DEFAULT_TYPE_REF,
- nullability: default_nullability,
+ nullability: r#type::Nullability::Unspecified as i32,
})),
})),
};
@@ -1419,7 +1419,7 @@ fn to_substrait_type(dt: &DataType, nullable: bool) ->
Result<substrait::proto::
parameter:
Some(parameter::Parameter::DataType(substrait::proto::Type {
kind: Some(r#type::Kind::I64(r#type::I64 {
type_variation_reference: DEFAULT_TYPE_REF,
- nullability: default_nullability,
+ nullability: r#type::Nullability::Unspecified as i32,
})),
})),
};
@@ -1443,7 +1443,7 @@ fn to_substrait_type(dt: &DataType, nullable: bool) ->
Result<substrait::proto::
kind: Some(r#type::Kind::UserDefined(r#type::UserDefined {
type_reference,
type_variation_reference: DEFAULT_TYPE_REF,
- nullability: default_nullability,
+ nullability,
type_parameters,
})),
})
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
