-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[Variant] variant_get tests clean up #9517
Copy link
Copy link
Closed
Labels
enhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelogparquet-variantparquet-variant* cratesparquet-variant* crates
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Reading changes in #9498 I noticed that the macros mention not using variant_shred because it's not implemented yet.
Describe the solution you'd like
- Use
variant_shredin these macros to make the code easier to read. - Add some
VariantArray::from_parts()while we're at it.
Describe alternatives you've considered
Additional context
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelogparquet-variantparquet-variant* cratesparquet-variant* crates