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

Ready to run

npx skills add https://github.com/apache/datafusion-comet --skill review-comet-pr

💬 Prompt for ChatGPT / Claude Chat▼

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.