harshmotw-db commented on code in PR #7783:
URL: https://github.com/apache/arrow-rs/pull/7783#discussion_r2167844189
##########
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:
It's possible some projects might prioritize performance and might not want
the Decimal type at all. I would prefer doing that as a follow up though. If
you agree, I'll go ahead and create an issue.
--
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
