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 Oct 13, 2021
1 parent 6eeb772 commit 89d214b
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 @@ -360,6 +360,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 @@ -490,6 +493,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 89d214b

Please sign in to comment.