apache/pinot

Add sanity check into Pinot Components start() method to ensure data dir is accessible

Open

#15860 opened on May 21, 2025

View on GitHub
 (3 comments) (0 reactions) (1 assignee)Java (4,937 stars) (1,234 forks)batch import
enhancementgood first issuehelp wanted

Description

During the component start, create and delete a tmp file into the data dir then delete it.

This is to ensure the permission is set correctly and fail earlier with a more explicit error.

Ref: https://apache-pinot.slack.com/archives/C011C9JHN7R/p1747663133588709

Contributor guide

Tech stack
java
Domain
backend
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Basic JavaFamiliarity with Pinot component lifecycleFile I/O
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
Examine the start() method in Pinot component classes (e.g., PinotEnvironmentProvider). Implement a check that creates a temporary file in the data directory and deletes it. Handle file permission errors gracefully. Reference the Slack thread for context.