sourcegraph/sourcegraph-public-snapshot

insights: replace/remove `GroupedResult` struct for compute streaming results

Aperta

#38.472 aperta il 8 lug 2022

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Go (1374 fork)auto 404
backendcode-insightsdebtgood first issue

Metriche repository

Star
 (10.290 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

PR #38416 removes deprecated code paths from the insights backend which means GroupedResult which we used for aggregating graphql results and adjacent structs are not needed anymore.

we can remove the computeTabulationResultToGroupedResults function and find a better common struct for ComputeTabulationResult, if any is needed

Guida contributor

Direzione di ricerca
Esamina la struttura `GroupedResult` deprecata e il suo utilizzo, quindi sostituiscila con una struttura comune o rimuovi la funzione.
Tech stack
go
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
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60