protocolbuffers/protobuf

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

Open

#17.085 aberto em 10 de jun. de 2024

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (16.128 forks)batch import
help wanted

Métricas do repositório

Stars
 (71.223 stars)
Métricas de merge de PR
 (Mesclagem média 2d 11h) (185 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Localize o método MapField.Add no código fonte C# do protobuf e modifique a ArgumentException para incluir o nome da chave duplicada.
Pilha de tecnologia
csharp
Domain
backendapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90