prestodb/presto

Use @TypeParameter to simplify multimap_agg aggregation function

Open

#21.364 aperta il 10 nov 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)Java (5240 fork)batch import
functiongood first issue

Metriche repository

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

Descrizione

With the merge of #21316 we can now simplify this aggregation function implementation by using @TypeParameter annotations in the serializer and state factory constructors, which means the function can be entirely converted to using the function annotation framework. This will simplify the implementation and improve maintainability.

Guida contributor

Direzione di ricerca
Studia l'uso dell'annotazione @TypeParameter da #21316 e applicala ai costruttori del serializzatore e della fabbrica di stato della funzione multimap agg, sostituendo la gestione manuale dei tipi.
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.
1-3 ore
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
JavaPresto function annotation framework
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65