alamb commented on PR #7783:
URL: https://github.com/apache/arrow-rs/pull/7783#issuecomment-3033652862

   > @alamb Do you think it is worth creating an arrow utility that abstracts 
away string to variant conversion? i.e. a function that takes in a StringArray 
(array of JSON strings some of which could be null), and returns a StructArray 
(array of corresponding variants where nulls are propagated directly)? If so, 
where should such functionality be in the codebase?
   
   Yes I think that is an important function
   
   I suggest calling it a `kernel` and putting it in a 
`parquet-variant-compute` crate
   
   Perhaps an API like this:
   
   ```rust
   fn to_variant(array: &Array) -> StructArray {
   ...
   }
   ```
   
   
   
   > 
   > Also, adding this high-level functionality would also abstract away many 
of the changes we make to the lower-level library. If we make changes to 
`VariantBuilder` to require buffers from the caller, we could simply modify 
this high-level function and users' workflows would stay the same.
   
   This is a great idea
   
   


-- 
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

Reply via email to