jecsand838 commented on code in PR #8220:
URL: https://github.com/apache/arrow-rs/pull/8220#discussion_r2302217663
##########
arrow-avro/src/reader/record.rs:
##########
@@ -736,6 +858,166 @@ fn sign_extend_to<const N: usize>(raw: &[u8]) ->
Result<[u8; N], ArrowError> {
Ok(arr)
}
+/// Lightweight skipping decoder for writer-only fields
Review Comment:
That's a good callout! I can definitely see where the confusion stems from.
I just updated the comment to read like this:
```
/// Lightweight skipper for non‑projected writer fields
/// (fields present in the writer schema but omitted by the
reader/projection);
/// per Avro 1.11.1 schema resolution these fields are ignored.
///
/// <https://avro.apache.org/docs/1.11.1/specification/#schema-resolution>
```
Let me know if that's more clear. I fully agree that comments /
documentation need to be straightforward and consistent in terminology and
language across the project.
--
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
