scovich commented on code in PR #9097:
URL: https://github.com/apache/arrow-rs/pull/9097#discussion_r2665628079
##########
arrow-json/src/reader/mod.rs:
##########
@@ -313,10 +314,21 @@ impl ReaderBuilder {
let num_fields = self.schema.flattened_fields().len();
+ // Extract projection field set from schema for projection-aware
parsing
+ // - strict_mode: Disabled (unknown fields cause errors anyway)
Review Comment:
If we're anyway doomed, shouldn't we go ahead and create (and then check)
the projection list? That way we'd fail-fast as soon as we first encounter the
unknown field, instead of building up the whole tape and leaving the the array
builders to fail later?
--
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]
