apache/pinot

Add read APIs for `BOOLEAN` and `TIMESTAMP`

Ouverte

#9 418 ouverte le 16 sept. 2022

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (2 personnes assignées)Java (1 234 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (4 937 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Currently BOOLEAN is supported using the INT data type, and TIMESTAMP is supported using the LONG data type. Even though the storage format are the same, the behavior should be different. E.g. reading BOOLEAN as STRING should give "true" or "false" instead of 1 or 0; reading DOUBLE as BOOLEAN should give doubleVal != 0 instead of (int) doubleVal. In order to achieve the expected behavior, we need to add read APIs for these 2 data types. It should be similar to how the BIG_DECIMAL is handled (storage format is BYTES). We want to add type support in:

  • Dictionary
  • ForwardIndexReader
  • BlockValSet
  • TransformFunction

Guide contributeur

Direction de recherche
Étudiez comment BIG DECIMAL est traité comme référence, car son format de stockage est BYTES mais il dispose d'API de lecture personnalisées. Implémentez des API similaires pour BOOLEAN et TIMESTAMP dans les classes Dictionary, ForwardIndexReader, BlockValSet et TransformFunction. Assurez vous que le cast en STRING renvoie des valeurs appropriées (par exemple, 'true'/'false' pour BOOLEAN). Les responsables actuels peuvent avoir un travail partiel ; vérifiez les commentaires de l'issue pour les mises à jour.
Stack technique
java
Domaine
backend
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.
2
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
JavaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75