yugabyte/yugabyte-db

[Platform] Leaks of resources

Open

#6 586 ouverte le 8 déc. 2020

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)C (1 003 forks)batch import
area/platformgood first issue

Métriques du dépôt

Stars
 (8 229 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (92 PRs mergées en 30 j)

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).

Guide contributeur

Direction de recherche
Examinez chaque fichier Java liste et ajoutez un nettoyage approprie des ressources en utilisant try with resources ou des blocs finally pour fermer les clients, lecteurs et ecrivains.
Stack technique
java
Domaine
backenddatabase
Type d'issue
Bug
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Java
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80