brancz opened a new pull request, #7929: URL: https://github.com/apache/arrow-rs/pull/7929
# Which issue does this PR close? Does not yet close, but contributes towards: - https://github.com/apache/arrow-rs/issues/6356 - https://github.com/apache/arrow-rs/issues/5981 - https://github.com/apache/arrow-rs/issues/1206 # Rationale for this change See the above issues. And this is a follow up to - https://github.com/apache/arrow-rs/pull/6711 - https://github.com/apache/arrow-rs/pull/6873 This was also split out from: https://github.com/apache/arrow-rs/pull/7467 # What changes are included in this PR? The required changes, so that nothing depends on the canonical `Schema`'s `Field` containing the `dict_id` field anymore, so that as a follow up, the `dict_id` field can actually be removed from the deprecated function signatures and remove the field itself. # Are these changes tested? All tests continue to pass. # Are there any user-facing changes? The function signatures of these publicly facing APIs changed to provide the appropriate access to the dict ID as it is represented in the respective IPC message(s): * flight_data_to_arrow_batch (arrow-flight) * arrow_data_from_flight_data (arrow-flight; sql) * read_record_batch (arrow-ipc) * read_dictionary (arrow-ipc) * FileDecoder::new (arrow-ipc) @tustvold @alamb @thinkharderdev @adriangb -- 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
