apache/pinot

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

Open

#15.860 geöffnet am 21. Mai 2025

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Java (4.937 Stars) (1.234 Forks)batch import
enhancementgood first issuehelp wanted

Beschreibung

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
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic JavaFamiliarity with Pinot component lifecycleFile I/O
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.