We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ScalarValue::FixedSizeBinary
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
@maxburke's PR added ScalarFixedSizeBinary support: #3911
ScalarFixedSizeBinary
However, it did not implement the serialization code to send ScalarValue::FixedSizeBinary over the network
Describe the solution you'd like
Fill out the code in datafusion/proto/src/to_proto.rs
scalar::ScalarValue::FixedSizeBinary(_, _) => { return Err(Error::General( "FixedSizeBinary is not yet implemented".to_owned(), )) }
And add a test for round trip serialization
Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.
Additional context Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered:
Should be a good first issue as there are plenty of other examples to follow along
Sorry, something went wrong.
It looks interesting. Can you assign it to me?
See also- #3612 - the serialization code is a bit of a mess now but hopefully we can get it better
retikulum
Successfully merging a pull request may close this issue.
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
@maxburke's PR added
ScalarFixedSizeBinary
support: #3911However, it did not implement the serialization code to send
ScalarValue::FixedSizeBinary
over the networkDescribe the solution you'd like
Fill out the code in datafusion/proto/src/to_proto.rs
And add a test for round trip serialization
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: