should remove targetFile when targetFile is existing?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript, node.js
- Ambito
- backend
Direzione di ricerca
Inizia tracciando il percorso di rollover che chiama fs.move(sourceFilePath, targetFilePath, { overwrite: true }) e riproduci il caso multiprocesso con disableClustering: true. Verifica il comportamento quando targetFilePath esiste già; il lavoro è completato quando i rollover concorrenti non sovrascrivono più il log rinnovato né causano la perdita di log.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
when the current file should roll,the operation is
fs.move(sourceFilePath, targetFilePath, { overwrite: true });
this will delete target file if it exist.
But, in the mutli process app,when set disableClustering: true, all process will do the same operation. The first write log process will rename log.log to log.log.2020-01-01-01 and renew log.log. The next write log process will rename log.log that is renewed by first process to log.log.2020-01-01-01 too and this cause log loss.
So, when target file exists, should skip remove operation?
try {
if (fs.existsSync(targetFilePath)) {
return ;
}
await fs.move(sourceFilePath, targetFilePath, { overwrite: true });
} catch (e) { }
- Lingua principale
- JavaScript
- Stelle
- 38
- Fork
- 20
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di log4js-node/streamroller
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
log4js-node/streamroller#176 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
log4js-node/streamroller#175 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
log4js-node/streamroller#173 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 15/100
log4js-node/streamroller#171 · 1 commento ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
log4js-node/streamroller#163 · 3 commenti ·
Tutte le issue di log4js-node/streamroller
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
mksglu/context-mode#1200 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
neondatabase/website#5944 ·
-
module: core
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
bigbluebutton/bigbluebutton#25849 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
jaegertracing/jaeger-ui#4506 ·