skills.rest
apache
apacheOFFICIAL
Explore all skills by this author →
WebsiteGitHub

review-comet-pr

Guide structured review of DataFusion Comet PRs for Spark compatibility.

1.2k|344|Updated Jan 15, 2024
Search Tags:#compatibility#review#guidelines#pr#spark#datafusion#comet
One-click install
npx skills add https://github.com/apache/datafusion-comet --skill review-comet-pr
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: review-comet-pr
Source: https://github.com/apache/datafusion-comet/tree/main/.claude/skills/review-comet-pr
Command: npx skills add https://github.com/apache/datafusion-comet --skill review-comet-pr

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides a structured, hands-off guide to review a DataFusion Comet pull request. It helps reviewers understand scope, gather context, and deliver consistent, high-quality feedback without posting comments directly.

Core Features & Use Cases

  • Structured review workflow: Outlines steps to gather PR metadata, read existing discussions, and plan feedback.
  • Consistency and quality checks: Guides verification of Spark compatibility, implementation correctness, and alignment with project guidelines.
  • Review orchestration: Includes recommended sequences for exploring PR diffs, Spark source references, and test coverage to support a robust review.

Quick Start

Follow the workflow to review a Comet PR by fetching its details and applying the provided guidance.

Frequently Asked Questions about review-comet-pr

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review a pull request for Spark compatibility?▼

To review a pull request for Spark compatibility, follow a structured workflow to gather PR metadata, read discussions, and cross-check diffs against Spark source code and tests. This verifies implementation correctness and alignment with project guidelines.

What is the best way to check DataFusion Comet PRs for implementation correctness?▼

The best way to check DataFusion Comet PRs for implementation correctness is to apply a structured review guide that verifies Spark compatibility, examines test coverage, and checks CI status against the project's contribution guidelines.

Does the Comet PR review process check against Spark source and tests?▼

Yes, the Comet PR review process includes a recommended sequence for exploring PR diffs, cross-checking Spark source references, and verifying test coverage to ensure robust Spark compatibility and implementation correctness.

Can I use a structured workflow to review Comet PRs modifying expressions or performance?▼

Yes, you can use a structured workflow to review Comet PRs that modify expressions, performance, or compatibility layers, guiding you to gather context and deliver consistent feedback without posting comments directly.

Why should I use a hands-off guide for Spark compatibility PR reviews?▼

You should use a hands-off guide for Spark compatibility PR reviews to ensure consistent, high-quality feedback by orchestrating the exploration of diffs, Spark source references, and test coverage without automatically posting comments.

Related Skills

Explore More Software Engineering Skills→
ohyesgocoolohyesgocool

ticket-intake

Parse Jira issues into structured briefs and numbered requirements lists.

Community
Advanced
apacheapache
1.2k

audit-comet-expression

Audit Comet expressions for correctness and test coverage across Spark versions.

Official
Advanced
AnveioAnveio
1

role-code-reviewer

Review pull requests for correctness, architecture, and maintainability.

Community
Intermediate
camilooscargbaptistacamilooscargbaptista
7

code-review

Route code review requests to specialized review skills by technology context.

Community
Advanced