prestodb/presto

OutputMatcher should take a Collection rather than a List

Aperta

#22.756 aperta il 15 mag 2024

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)Java (5240 fork)batch import
available-to-pickupgood first issueplanner

Metriche repository

Star
 (15.558 stelle)
Metriche merge PR
 (Merge medio 34g 14h) (120 PR mergiate in 30 g)

Descrizione

com.facebook.presto.sql.planner.assertions.OutputMatcher should take a Collection rather than a List

Order is not important in this class so this is semantically correct, and I encountered a case where it would have been mildly more convenient to pass in a Set instead of a List.

Guida contributor

Direzione di ricerca
Trova la classe OutputMatcher e cambia il suo parametro del metodo da List a Collection. Aggiorna tutti i chiamanti che passano una List ad accettare una Collection. Assicurati che nessun altro codice dipenda dall'ordine.
Tech stack
java
Dominio
backenddatabase
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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.
85