Skip to content

Add examples of using Field::try_extension_type#8475

Merged
alamb merged 2 commits intoapache:mainfrom
alamb:alamb/extension_type_docs
Sep 28, 2025
Merged

Add examples of using Field::try_extension_type#8475
alamb merged 2 commits intoapache:mainfrom
alamb:alamb/extension_type_docs

Conversation

@alamb
Copy link
Copy Markdown
Contributor

@alamb alamb commented Sep 26, 2025

Which issue does this PR close?

We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax.

Rationale for this change

The implications of using Field::try_extension_type may not be clear, so let's document it better. See #8474 for more background

What changes are included in this PR?

Add some clarifying comments and a doc example

Here is an example of it rendered
Screenshot 2025-09-26 at 12 14 31 PM

Are these changes tested?

Yes new doc example tets

Are there any user-facing changes?

No API / code changes

Copy link
Copy Markdown
Member

@mbrobbel mbrobbel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great addition, thanks @alamb

@alamb alamb merged commit 0737c61 into apache:main Sep 28, 2025
27 checks passed
@alamb alamb deleted the alamb/extension_type_docs branch September 28, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrow Changes to the arrow crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants