protocolbuffers/protobuf

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

Open

#17 085 ouverte le 10 juin 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (16 128 forks)batch import
help wanted

Métriques du dépôt

Stars
 (71 223 stars)
Métriques de merge PR
 (Merge moyen 2j 11h) (185 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Localisez la methode MapField.Add dans le code source C# de protobuf et modifiez l'ArgumentException pour inclure le nom de la cle dupliquee.
Stack technique
csharp
Domaine
backendapi
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90