Skip to content

GH-47157: [Docs] Improve presentation of Other available packages section in build_system.rst - #47411

Merged
amoeba merged 4 commits into
apache:mainfrom
vidhyarbiji:doc-update-other-package
Aug 28, 2025
Merged

GH-47157: [Docs] Improve presentation of Other available packages section in build_system.rst#47411
amoeba merged 4 commits into
apache:mainfrom
vidhyarbiji:doc-update-other-package

Conversation

@vidhyarbiji

@vidhyarbiji vidhyarbiji commented Aug 22, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Rationale for this change

The table in the "Other available packages" section of build_system.rst was refactored for improved readability.

What changes are included in this PR?

Refactored the table in the "Other available packages" section of build_system.rst

Are these changes tested?

Verified the preview of docs

Are there any user-facing changes?

No.

@vidhyarbiji

vidhyarbiji commented Aug 27, 2025

Copy link
Copy Markdown
Contributor Author

@amoeba, could you please help to review. Thanks.

@amoeba

amoeba commented Aug 27, 2025

Copy link
Copy Markdown
Member

@github-actions crossbow submit preview-docs

@github-actions

github-actions Bot commented Aug 27, 2025

Copy link
Copy Markdown

Revision: 5a00b71

Submitted crossbow builds: ursacomputing/crossbow @ actions-527fbaa1ae

Task Status
preview-docs GitHub Actions

amoeba
amoeba requested changes Aug 28, 2025
View reviewed changes

@amoeba amoeba 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.

Thanks for taking this on, this new format looks great. I had a few minor suggestions, let me know what you think.

Comment thread docs/source/cpp/build_system.rst Outdated
Comment thread docs/source/cpp/build_system.rst Outdated
Comment thread docs/source/cpp/build_system.rst Outdated
@amoeba

amoeba commented Aug 28, 2025

Copy link
Copy Markdown
Member

Current draft (with my suggestions) in case anyone else wants to be saved a click: http://crossbow.voltrondata.com/pr_docs/47411/cpp/build_system.html#other-available-packages.

@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Aug 28, 2025
vidhyarbiji and others added 3 commits August 28, 2025 09:54
@vidhyarbiji @amoeba
Update Other available packages section in docs/source/cpp/build_syst…
313e719 
…em.rst

Co-authored-by: Bryce Mecum <petridish@gmail.com>
@vidhyarbiji @amoeba
Update Other available packages section in docs/source/cpp/build_syst…
441f7a2 
…em.rst

Co-authored-by: Bryce Mecum <petridish@gmail.com>
@vidhyarbiji @amoeba
Update Other available packages section in docs/source/cpp/build_syst…
6e8b174 
…em.rst

Co-authored-by: Bryce Mecum <petridish@gmail.com>
@vidhyarbiji
vidhyarbiji requested a review from amoeba August 28, 2025 04:26
@amoeba

amoeba commented Aug 28, 2025

Copy link
Copy Markdown
Member

@github-actions crossbow submit preview-docs

@github-actions

github-actions Bot commented Aug 28, 2025

Copy link
Copy Markdown

Revision: 6e8b174

Submitted crossbow builds: ursacomputing/crossbow @ actions-db71aa1b95

Task Status
preview-docs GitHub Actions

amoeba
amoeba approved these changes Aug 28, 2025
View reviewed changes

@amoeba amoeba 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

@amoeba
amoeba merged commit ed77d25 into apache:main Aug 28, 2025
7 checks passed
@amoeba amoeba removed the awaiting committer review Awaiting committer review label Aug 28, 2025
@amoeba amoeba mentioned this pull request Aug 28, 2025
Closed
@amoeba

amoeba commented Aug 28, 2025

Copy link
Copy Markdown
Member

Thanks @vidhyarbiji!

@conbench-apache-arrow

conbench-apache-arrow Bot commented Aug 29, 2025

Copy link
Copy Markdown

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

There weren't enough matching historic benchmark results to make a call on whether there were regressions.

The full Conbench report has more details.

@rmnskb rmnskb mentioned this pull request Oct 7, 2025
Merged
Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
@vidhyarbiji @amoeba @Mottl
apacheGH-47157: [Docs] Improve presentation of Other available packag…
e29025a 
…es section in build_system.rst (apache#47411)

### Rationale for this change

The table in the "Other available packages" section of build_system.rst was refactored for improved readability.

### What changes are included in this PR?

Refactored the table in the "Other available packages" section of build_system.rst

### Are these changes tested?

Verified the preview of docs

### Are there any user-facing changes?

No.

* GitHub Issue: apache#47157

Lead-authored-by: Vidhya Ravikumar <vidhyarbiji@gmail.com>
Co-authored-by: vidhya.ravikumar <vidhya.ravikumar@ibm.com>
Co-authored-by: Bryce Mecum <petridish@gmail.com>
Signed-off-by: Bryce Mecum <petridish@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@amoeba amoeba amoeba approved these changes

Assignees

No one assigned

Labels

Component: Documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vidhyarbiji @amoeba