sourcegraph/sourcegraph-public-snapshot

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

Ouverte

#38 472 ouverte le 8 juil. 2022

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Go (1 374 forks)auto 404
backendcode-insightsdebtgood first issue

Métriques du dépôt

Stars
 (10 290 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Examinez la structure `GroupedResult` dépréciée et son utilisation, puis remplacez la par une structure commune ou supprimez la fonction.
Stack technique
go
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60