trinodb/trino

Update presto-record-decoder and dependent modules to use Optional

Aperta

#1610 aperta il 30 set 2019

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Java (2678 fork)batch import
good first issuemaintenance

Metriche repository

Star
 (9113 stelle)
Metriche merge PR
 (Merge medio 5g 9h) (306 PR mergiate in 30 g)

Descrizione

Variables such as getFormatHint in DecoderColumnHandle can be null, we should change them to optional

Guida contributor

Direzione di ricerca
Trova tutti gli usi di getFormatHint nel modulo presto record decoder e nei suoi dipendenti, cambia la firma del metodo per restituire Optional<FormatHint> invece di FormatHint nullable, e aggiorna tutti i chiamanti di conseguenza.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
JavaOptional API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70