apache/pinot

Increase code coverage

Open

#5.150 aberto em 13 de mar. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Java (1.234 forks)batch import
good first issue

Métricas do repositório

Stars
 (4.937 stars)
Métricas de merge de PR
 (Mesclagem média 6d 7h) (186 fundiu PRs em 30d)

Description

Currently the code coverage of pinot repo is just around 50% (Link). This issue tracks the status of increasing code coverage.

The following modules currently have less than 50% code coverage or don't have any test at all:

  1. pinot-clients/pinot-java-client (46.15%)
  2. pinot-common (47.96%)
  3. pinot-plugins (36.38%) No code coverage:
  • pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-common
  • pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-hadoop
  • pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-spark
  • pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-stadardalone
  • pinot-plugins/pinot-file-system/pinot-gcs
  • pinot-plugins/pinot-file-system/pinot-hdfs

The rest of the modules shown below seems fine but we should still bring them to at least 75%. pinot-broker (76.6%) pinot-controller (63.29%) pinot-core (76.85%) pinot-minion (68.98%) pinot-server (67.05%) pinot-spi (68.84%)

Guia do colaborador

Direção de pesquisa
Escolha um módulo com baixa cobertura de código (ex: pinot java client) e escreva testes unitários para suas classes não testadas. Consulte os testes existentes em outros módulos para estilo e estrutura.
Pilha de tecnologia
java
Domain
backenddata
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 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.
Claro
Prerequisites
JavaJUnit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60