envoyproxy/envoy

Make wrapped / native scalar types consistent in v4 API

Open

#8.128 aberto em 3 de set. de 2019

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
api/v4help wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

In v2, we started of by wrapping everything, but then settled on a policy of only wrapping scalar fields that have non-proto3 defaults. There is a legacy of this across the API that should be cleaned up to be more consistent, v3 is such an opportunity.

Guia do colaborador

Direção de pesquisa
A issue exige tornar os tipos escalares empacotados (por exemplo, google.protobuf.BoolValue) e nativos consistentes em toda a API v3. Revise os comentários da issue em https://github.com/envoyproxy/envoy/issues/8128 para quaisquer discussões sobre políticas. Pesquise no código fonte do Envoy (especialmente arquivos .proto) todos os usos de tipos escalares empacotados e compare com as contrapartes nativas. Proponha uma política consistente e identifique os arquivos que precisam de alterações. A issue é grande e requer um entendimento profundo do Protobuf e do design da API do Envoy.
Pilha de tecnologia
cpp
Domain
apibackend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
GitC++Protobuf
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45