sourcegraph/sourcegraph-public-snapshot

insights: refactor capture group methods for readability and performance

Aperta

#33.148 aperta il 28 mar 2022

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

Metriche repository

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

Descrizione

When a capture group insight is created, we tally up matches per capture group using MatchValues and Counts methods on ComputeMatchContext. We then group them in the method GroupByCaptureMatch.

The current logic uses many nested for loops. The aim of this ticket is to look into simplifying this logic. What we want at the end is a map of unique capture group values to their count.

Guida contributor

Direzione di ricerca
Ispeziona i metodi MatchValues, Counts e GroupByCaptureMatch nel codebase di sourcegraph. Identifica i cicli annidati ed esplora alternative come l'uso di una mappa per aggregare i conteggi direttamente o l'uso di stream per ridurre la complessità.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60