etseidl commented on code in PR #8528:
URL: https://github.com/apache/arrow-rs/pull/8528#discussion_r2395468308
##########
parquet/src/basic.rs:
##########
@@ -928,17 +930,31 @@ enum BoundaryOrder {
// ----------------------------------------------------------------------
// Mirrors thrift enum `EdgeInterpolationAlgorithm`
+// TODO(ets): we need to allow for unknown variants. Either hand code this
one, or add a new
+// macro that adds an _Unknown variant.
Review Comment:
If stats are robust to unknown algorithms, then I think we should add an
`_Unknown` variant here so older readers can still handle newer files. I'll
make that change soon.
--
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]
