apache/pinot

Introduce reason codes in server http requests to controller

Open

#12 055 ouverte le 28 nov. 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Java (1 234 forks)batch import
cleanupgood first issue

Métriques du dépôt

Stars
 (4 937 stars)
Métriques de merge PR
 (Merge moyen 6j 7h) (186 PRs mergées en 30 j)

Description

pinot servers send HTTP requests to the controller for posting a completed realtime segment, posting an indication that realtime segments have stopped consuming, or that the consumer could never be created.

Currently there is a reason string that is included as an argument. This is error prone. A reason code makes it easier to incorporate code for appropriate action, given a particular reason.

The con side of a reason code is that it is a bit harder to evolve. We need to be careful writing code on the controller side, since servers may send unknown reason codes. As long as we follow some careful practices, introducing reason codes should get us to a better place.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle des requêtes HTTP du serveur Pinot au contrôleur pour l'achèvement du segment en temps réel et identifiez où la chaîne de raison est utilisée. Proposez une énumération de codes de raison et adaptez le contrôleur pour gérer les codes inconnus avec élégance.
Stack technique
java
Domaine
backenddatabasedistributed systems
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaHTTPDistributed Systems
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60