jetty/jetty.project

Review RolloverFileOutputStream locking

Open

#5092 aperta il 29 lug 2020

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)Java (1913 fork)batch import
Help WantedLow PriorityPinned

Metriche repository

Star
 (3701 star)
Metriche merge PR
 (Merge medio 6g 14h) (48 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Inizia esaminando la classe `RolloverFileOutputStream`, probabilmente nel pacchetto `org.eclipse.jetty.util.log`. Concentrati sul campo ` rollover` e su come viene utilizzato nel costruttore e nell'attività di rollover pianificata. L'obiettivo è rimuovere il blocco a livello di classe non necessario e introdurre una sincronizzazione minima tra le scritture e l'attività di rollover per prevenire la perdita di righe di log. Osserva come il file viene aperto e ruotato, e considera l'uso di un oggetto di blocco privato o operazioni atomiche. Controlla l'issue collegato #5083 per ulteriore contesto.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Java
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50