asubiotto commented on code in PR #12071:
URL: https://github.com/apache/arrow-rs/pull/12071#discussion_r4479971817
##########
arrow-ipc/src/reader.rs:
##########
@@ -915,6 +915,41 @@ fn update_dictionaries(
Ok(())
}
+/// Returns the first field with dictionary `id`, in
[`Schema::fields_with_dict_id`] order.
+///
Review Comment:
It seems like nobody within arrow calls this other than to get the first
match *but* it is a breaking API change, so would need a major release. It's
also marked deprecated so I guess should be removed at some point.
--
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]
