alamb commented on issue #7870:
URL: https://github.com/apache/arrow-rs/issues/7870#issuecomment-3038882242
> ... that could work, but only if it also creates one metadata dictionary
per variant value and tracks those offsets as well:
We probably want to make sure we can share the metadata across rows if
desired (so in other words I don't think it should require starting a new
metadata...)
I suggest we postpone any more API changes here until we start working on
actual variant cast kernels 🤔
Maybe we could start with one we know we will likely want
```rust
/// Converts a StringArray, LargeStringArray or StringViewArray that contains
/// JSON formatted text into a Struct array containing a Parquet Variant
json_to_variant_array(arr: ArrayRef) -> Result<StructArray> {
...
}
```
I think that will also force us to figure out what kind of arrays will be
supported
For example is it
```rust
StructArray {
metadata: BinaryView,
value: BinaryView
}
```
Or
```rust
StructArray {
metadata: Binary,
value: Binary
}
```
Or something else
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
