Skip to content
cccskills
FreePublished on GitHub

pr_review

Review Apache DataFusion pull requests using the project's review guide. Gather the proposed changes, discussion, and automated check results, or apply the same review criteria to your own changes before submitting a pull request.

OriginalShow the original description

Review Apache DataFusion pull requests following the project's PR review guide. Use whenever asked to review a DataFusion PR or PR URL, and whenever creating a PR, to check the changes against the same criteria before submitting.

Works withGitHub
See how to install

Good for

  • Reviewing an Apache DataFusion pull request
  • Checking your changes before submitting a pull request
  • Reviewing changes alongside discussion and automated check results

Files included (1)

  • SKILL.md1.2 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

DataFusion PR Review

This skill describes the mechanics for doing PR reviews from the command line.

When creating a PR, skip the "Collect PR context" step and instead check the changes against each area of the PR review guide before submitting.

Collect PR context

  • Check out the PR locally: gh pr checkout <PR number> (ask first if the working tree has other work in progress).
  • Fetch the PR description, comments, and reviews: gh pr view <PR number> --json title,body,comments,reviews
  • Fetch CI status: gh pr checks <PR number>.

Compute the diff

# find the remote that points at apache/datafusion (e.g. `apache`, `upstream`, or `origin`)
UPSTREAM=$(git remote -v | grep -m1 'apache/datafusion' | cut -f1)
git fetch $UPSTREAM main
git diff $(git merge-base HEAD $UPSTREAM/main)

Review checklist

Work through each area from the PR review guide.

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

add-benchmarks

FreeSummary

Design or extend DataFusion benchmarks using SQL where practical. Select representative workload variations, keep unrelated processing lightweight, and integrate runs with bench.sh so measurements reflect useful execution paths and remain practical to run.

  • Add SQL benchmarks for DataFusion operators
  • Select representative input sizes and filter conditions
  • Reduce unrelated processing overhead in measurements
apache/datafusion9.5KUpdated Oct 17, 2026

Audit one datafusion-spark function against Apache Spark 4.2.0. Compare source code, accepted types, and edge cases, verify expected results with PySpark, then fix differences and record remaining gaps as issues and disabled tests.

  • Auditing a function for Spark compatibility
  • Checking NULL, overflow, and ANSI mode behavior
  • Finding missing SQL test coverage
apache/datafusion9.5KUpdated Oct 17, 2026

datafusion-ffi

FreeSummary

Guide changes and reviews in datafusion-ffi, which connects separately compiled libraries. Check binary compatibility, memory ownership, and method coverage, and choose unit and cross-library tests for new wrappers, trait extensions, or codec changes.

  • Add FFI wrappers for DataFusion traits
  • Check that default method overrides cross the FFI boundary
  • Review binary compatibility and memory ownership in pull requests
apache/datafusion9.5KUpdated Oct 17, 2026

See all skills by apache

Report a problem with this skill