apache/pinot

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

Open

#15.860 aberto em 21 de mai. de 2025

Ver no 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

Guia do colaborador

Pilha de tecnologia
java
Domain
backend
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
active
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic JavaFamiliarity with Pinot component lifecycleFile I/O
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
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.