Skip to content

GH-50285: [Docs] Update documentation to reflect C# improvements#50286

Merged
CurtHagenlocher merged 2 commits into
apache:mainfrom
CurtHagenlocher:csharp_docs
Jun 29, 2026
Merged

GH-50285: [Docs] Update documentation to reflect C# improvements#50286
CurtHagenlocher merged 2 commits into
apache:mainfrom
CurtHagenlocher:csharp_docs

Conversation

@CurtHagenlocher

@CurtHagenlocher CurtHagenlocher commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

What changes are included in this PR?

Updates documentation and integration tests to reflect improvements in the dotnet implementation.

Are these changes tested?

N/A

Are there any user-facing changes?

N/A

@CurtHagenlocher CurtHagenlocher requested a review from pitrou as a code owner June 29, 2026 15:14
Copilot AI review requested due to automatic review settings June 29, 2026 15:14
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #50285 has no components, please add labels for components.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Arrow “status” documentation and Archery JSON integration test data generation to reflect expanded support in the C#/.NET implementation.

Changes:

  • Update C# capability markers in docs/source/status.rst (e.g., Extension, Run-End Encoded, UUID, 8-bit Boolean, Parquet Variant, and Large List View via note reference).
  • Enable .NET to participate in specific generated JSON integration cases by removing .skip_tester('.NET') for Run-End Encoded and (Large) List View cases.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/source/status.rst Updates documented C# feature support matrix entries and note markers.
dev/archery/archery/integration/datagen.py Removes .NET skips so the corresponding integration JSON cases run for .NET.

Comment thread docs/source/status.rst Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 29, 2026 15:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@pitrou pitrou changed the title GH-50285: Update documentation to reflect C# improvements GH-50285: [Docs] Update documentation to reflect C# improvements Jun 29, 2026

@pitrou pitrou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, thank you @CurtHagenlocher

@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Jun 29, 2026
@CurtHagenlocher CurtHagenlocher merged commit d40ace0 into apache:main Jun 29, 2026
31 checks passed
@CurtHagenlocher CurtHagenlocher deleted the csharp_docs branch June 29, 2026 15:57
@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit d40ace0.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants