unisonweb/unison

switch `Patch` to use `Map` instead of `Relation`

Open

#1658 aperta il 17 ago 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Haskell (304 fork)batch import
help wanted

Metriche repository

Star
 (6624 star)
Metriche merge PR
 (Merge medio 10g 2h) (5 PR mergiate in 30 g)

Descrizione

We don't need the reverse index that Relation provides hanging out in memory.

We don't want to risk changing the existing V1 serialization format, so for simplicity, when constructing this PR, convert to/from a Relation using Relation.toMultimap/.fromMultimap before calling the serialization functions.

Guida contributor

Direzione di ricerca
Studia l'implementazione corrente di Patch e Relation, quindi sostituisci Relation con Map internamente, utilizzando Relation.toMultimap e Relation.fromMultimap per la compatibilità di serializzazione.
Tech stack
Nessuno
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
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75