alamb opened a new issue, #7775:
URL: https://github.com/apache/arrow-rs/issues/7775

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   * @carpecodeum added support for writing variants as JSON as part of 
https://github.com/apache/arrow-rs/pull/7670
   
   However, https://github.com/apache/arrow-rs/pull/7670 also introduced a 
dependency on `serde_json`. As the various arrow and parquet crates are often 
low in the dependency stack and widely used, we are trying to avoid unnecessary 
dependencies
   
   `serde_json` is a very helpful library for programmatically manipulating 
json objects, but many arrow usecases that use variant will not use this 
library (as the whole point of variant is to be more efficient)
   
   
   **Describe the solution you'd like**
   1. I would like the `parquet-varaint` crate to *NOT* depend on `serde_json` 
by default. 
   2. I would like `parquet-variant` to be a optional dependency -- so if 
people want easy / efficient `serde_json` conversion they can enable it
   
   So like 
   ```shell
   # by default, does not have serde_json support
   cargo test -p parquet-variant
   # if the `serde_json` feature is added, now the serde_json code is available
   cargo test -p parquet-variant --features=serde_json
   # or also with --all-features
   cargo test -p parquet-variant --all-features
   ```
   
   **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.
   -->
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to