Skip to content

GH-32503: [Docs][Benchmarking] Document pull request benchmark hooks - #50953

Open
nightcityblade wants to merge 1 commit into
apache:mainfrom
nightcityblade:docs/issue-32503
Open

GH-32503: [Docs][Benchmarking] Document pull request benchmark hooks#50953
nightcityblade wants to merge 1 commit into
apache:mainfrom
nightcityblade:docs/issue-32503

Conversation

@nightcityblade

@nightcityblade nightcityblade commented Aug 22, 2026

Copy link
Copy Markdown

Rationale for this change

Fixes #32503.

The developer benchmark guide explains local Archery runs but does not describe how pull requests use the remote benchmark service or repository-owned build hooks.

What changes are included in this PR?

  • Document the pull request comment that schedules remote benchmarks.
  • Explain how Conbench compares the contender and baseline commits.
  • Link the benchmark environment and hooks documentation and clarify that hook changes are tested from the pull request commit.

Are these changes tested?

  • uvx sphinx-lint docs/source/developers/benchmarks.rst
  • make -C docs dev with the documented requirements installed in an isolated environment
  • Inspected the rendered benchmarks page to verify the new section, command block, and link.

Are there any user-facing changes?

Yes. The developer documentation now explains pull request benchmarking and benchmark hook changes.

AI usage

This documentation update was drafted with Codex assistance. I reviewed the current workflow against recent Arrow pull request benchmark comments, checked every line of the change, and verified the rendered output locally.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #32503 has been automatically assigned in GitHub to PR creator.

@uros-b

uros-b commented Aug 24, 2026

Copy link
Copy Markdown
Member

Thank you @nightcityblade! LGTM

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.

[Docs][Benchmarking] Add info about the use of benchmarking hooks on the PR

2 participants