Is your feature request related to a problem or challenge?
Part of #11752
We are trying to enable DataFusion to use StringViewArray by default. If we do that it means ScalarValue::Utf8View and ScalarValue::BinaryView will be more likely to be used in plans
Describe the solution you'd like
Thus we need to ensure ScalarValue::Utf8View and ScalarValue::BinaryView can be serialized using datafusion-proto
Describe alternatives you've considered
I recommend adding coverage for ScalarValue::Utf8View and `ScalarValue::BinaryView to the tests here
|
ScalarValue::Utf8(Some(String::from("Test string "))), |
|
ScalarValue::LargeUtf8(Some(String::from("Test Large utf8"))), |
And then update the code to get the tests to pass
Additional context
No response
Is your feature request related to a problem or challenge?
Part of #11752
We are trying to enable DataFusion to use StringViewArray by default. If we do that it means
ScalarValue::Utf8ViewandScalarValue::BinaryViewwill be more likely to be used in plansDescribe the solution you'd like
Thus we need to ensure
ScalarValue::Utf8ViewandScalarValue::BinaryViewcan be serialized using datafusion-protoDescribe alternatives you've considered
I recommend adding coverage for
ScalarValue::Utf8Viewand `ScalarValue::BinaryView to the tests heredatafusion/datafusion/proto/tests/cases/roundtrip_logical_plan.rs
Lines 1233 to 1234 in cb1e3f0
And then update the code to get the tests to pass
Additional context
No response