protocolbuffers/protobuf

Dotnet - Include key name when failing to add to a MapField due to a duplicate key

Open

#17.085 aperta il 10 giu 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (16.128 fork)batch import
help wanted

Metriche repository

Star
 (71.223 star)
Metriche merge PR
 (Merge medio 2g 11h) (185 PR mergiate in 30 g)

Descrizione

What language does this apply to? C#/dotnet

Describe the problem you are trying to solve. Trying to understand why adding a key to a MapField fails

Describe the solution you'd like The Argument exception should contain the name of the key that failed being added https://github.com/protocolbuffers/protobuf/blob/54d8f03974c108ef8fd0f26568cd9eb086165568/csharp/src/Google.Protobuf/Collections/MapField.cs#L97

Describe alternatives you've considered None

Additional context We aren't using this library directly, but through a 3rd party library that uses Google.Cloud.Logging.V2, making debugging challenging

Guida contributor

Direzione di ricerca
Individua il metodo MapField.Add nel codice sorgente C# di protobuf e modifica l'ArgumentException per includere il nome della chiave duplicata.
Tech stack
csharp
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90