jetty/jetty.project

Review RolloverFileOutputStream locking

Open

#5 092 ouverte le 29 juil. 2020

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)Java (1 913 forks)batch import
Help WantedLow PriorityPinned

Métriques du dépôt

Stars
 (3 701 stars)
Métriques de merge PR
 (Merge moyen 6j 14h) (48 PRs mergées en 30 j)

Description

Spawned by #5083.

Jetty version 9.4.x

Description RolloverFileOutputStream locking should be improved/removed.

Currently it locks on a class lock for no reasons, since __rollover is only ever written in the constructor (and thus could be final).

There should be some synchronization between the writes and the scheduled task (so that the file can be swapped without losing log lines), but the class locks seem unnecessary.

Guide contributeur

Direction de recherche
Commencez par examiner la classe `RolloverFileOutputStream`, probablement dans le package `org.eclipse.jetty.util.log`. Concentrez vous sur le champ ` rollover` et sur la façon dont il est utilisé dans le constructeur et la tâche de rotation planifiée. L'objectif est de supprimer le verrouillage inutile au niveau de la classe et d'introduire une synchronisation minimale entre les écritures et la tâche de rotation pour éviter la perte de lignes de journal. Examinez comment le fichier est ouvert et rotationné, et envisagez d'utiliser un objet de verrouillage privé ou des opérations atomiques. Consultez l'issue liée #5083 pour plus de contexte.
Stack technique
java
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
50