Skip to content

Commit

Permalink
Add minimal ballista support
Browse files Browse the repository at this point in the history
  • Loading branch information
alamb committed Sep 9, 2021
1 parent b2cd759 commit f089217
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions ballista/rust/core/src/serde/logical_plan/to_proto.rs
Original file line number Diff line number Diff line change
Expand Up @@ -407,6 +407,9 @@ impl From<&DataType> for protobuf::arrow_type::ArrowTypeEnum {
fractional: *fractional as u64,
})
}
DataType::Map(_, _) => {
unimplemented!("Ballista does not yet support Map data type")
}
}
}
}
Expand Down Expand Up @@ -537,6 +540,7 @@ impl TryFrom<&DataType> for protobuf::scalar_type::Datatype {
| DataType::Struct(_)
| DataType::Union(_)
| DataType::Dictionary(_, _)
| DataType::Map(_, _)
| DataType::Decimal(_, _) => {
return Err(proto_error(format!(
"Error converting to Datatype to scalar type, {:?} is invalid as a datafusion scalar.",
Expand Down

0 comments on commit f089217

Please sign in to comment.