yugabyte/yugabyte-db

[Platform] Leaks of resources

Open

#6.586 aberto em 8 de dez. de 2020

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)C (1.003 forks)batch import
area/platformgood first issue

Métricas do repositório

Stars
 (8.229 stars)
Métricas de merge de PR
 (Mesclagem média 17d 21h) (92 fundiu PRs em 30d)

Description

  1. UniverseTaskBase.java, isServerAlive(), absent ybService.closeClient();
  2. Universe.java, getMasterLeader(), it is better to put closeClient() into the finally block;
  3. SetFlagInMemory.java, run(), closeClient() could be missed in case of any exceptions raised after getClient() and before closeClient();
  4. WaitForServerReady.java, run(), it is better to put closeClient() into the finally block. Yes, it is almost impossible that ServerSubTaskBase.getHostPort() will throw an exception but actually this probability is not 0;
  5. SessionController.java, getLogs(), Resource leak: 'reader' is never closed;
  6. SwamperHelper.java, writeTargetJsonFile(), the file may stay opened (in case of file.write of file.flush throws an exception);
  7. CertificateHelper.java, createRootCA(), Resource leak: 'certWriter' is never closed, Resource leak: 'keyWriter' is never closed; createClientCertificate(), 'clientCertWriter' and 'clientKeyWriter' are never closed (in case when storagePath != null).

Guia do colaborador

Direção de pesquisa
Examine cada arquivo Java listado e adicione a limpeza adequada de recursos usando try with resources ou blocos finally para fechar clientes, leitores e gravadores.
Pilha de tecnologia
java
Domain
backenddatabase
Tipo Issue
Bug
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Java
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80