apache/pinot

Add read APIs for `BOOLEAN` and `TIMESTAMP`

Open

#9418 aperta il 16 set 2022

Vedi su GitHub
 (5 commenti) (0 reazioni) (2 assegnatari)Java (1234 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (4937 star)
Metriche merge PR
 (Merge medio 6g 7h) (186 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Studia come BIG DECIMAL viene gestito come riferimento, poiché il suo formato di archiviazione è BYTES ma dispone di API di lettura personalizzate. Implementa API simili per BOOLEAN e TIMESTAMP nelle classi Dictionary, ForwardIndexReader, BlockValSet e TransformFunction. Assicurati che il cast a STRING restituisca valori appropriati (ad esempio, 'true'/'false' per BOOLEAN). Gli assegnatari esistenti potrebbero avere lavori parziali; controlla i commenti dell'issue per aggiornamenti.
Tech stack
java
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75