scovich commented on code in PR #7783:
URL: https://github.com/apache/arrow-rs/pull/7783#discussion_r2168001273


##########
parquet-variant/Cargo.toml:
##########
@@ -35,7 +35,10 @@ rust-version = "1.83"
 [dependencies]
 arrow-schema = { workspace = true }
 chrono = { workspace = true }
-serde_json = "1.0"
+# "arbitrary_precision" allows us to manually parse numbers. "preserve_order" 
does not automatically
+# sort object keys
+serde_json = { version = "1.0", features = ["arbitrary_precision", 
"preserve_order"] }

Review Comment:
   Oh, for sure it's a follow-up. But we probably do need a story for how to 
handle that feature flag in a dependency (maybe part of the bigger story of how 
to handle a serde_json dependency in the first place).



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