Skip to main content

review-correctness-nulls

Review Apache Pinot correctness when null handling, numeric types, dispatch, or resource lifecycles change.

Source facts

Repository
apache/pinot
Last source activity
September 13, 2026 at 20:06
Detected SKILL.md language
English
Stars
6,149
Forks
1,523

Install options

The review-first prompt is selected by default. You can switch to a direct command or download a local copy.

Review the source files

Read SKILL.md and any companion files shown by SkillsMP before deciding whether to install.

Showing SKILL.md

SKILL.md
Source instructions · Read-only preview
name
review-correctness-nulls
description
Review Apache Pinot correctness when null handling, numeric types, dispatch, or resource lifecycles change.
domain
kb/code-review-principles.md#5-correctness--safety
triggers
["diff touches null-vector / null-bitmap / null-enabled code paths","diff adds a switch on DataType, FieldSpec.DataType, or IndexType without default throw","diff changes arithmetic / aggregation / window function type dispatch","diff touches segment destroy / close / persist paths"]
license
Apache-2.0
# Skill: review-correctness-nulls Procedure: see [`kb/skills/review-correctness-nulls.md`](../../../kb/skills/review-correctness-nulls.md). Read it first, then follow it.
View on GitHub