Cleanup: conflict handling without checksum
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- javascript
- Ambito
- backend, distributed-systems
Direzione di ricerca
Start by reading lib/Listeners/NodeWrittenResetDocumentListener.php around line 58 and trace how checksum checks interact with file locks and conflict handling. Compare the listed alternatives, including the linked Nextcloud server issue; the issue does not settle which approach to take. Done means agreeing on and implementing a conflict-handling approach that avoids unnecessary full-file reads while preserving conflict detection.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
We currently store a checksum on every .md file save:
https://github.com/nextcloud/text/blob/19951abbbaca10923da261a74935ec08a85d2e7f/lib/Listeners/NodeWrittenResetDocumentListener.php#L58
This is for two reasons:
- Avoid extra conflict dialogs when the file is overwritten with the same content.
- Detect actual conflicts when the file is written twice in the same second.
Computing the checksum requires reading the entire file and can be costly for large files on external storage.
While this exists for historical reasons we now ship the files_lock app by default which prevents overwriting a text file that is being edited because it's locked.
Alternatives to the status quo
a) Rely on files_lock to prevent most conflicts. Trigger conflicts without checking checksum when the lock is broken or the file is written.
b) Use a server wide checksum implementation https://github.com/nextcloud/server/issues/56057
c) only check checksums when files_lock is not there
d) Rely on files_lock and update live editing sessions with the new content when a lock is broken.
- Lingua principale
- JavaScript
- Stelle
- 665
- Fork
- 133
- Merge medio
- 1g 17h
- PR unite (30g)
- 172
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 nextcloud/text
-
0. Needs triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
nextcloud/text#9482 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
0. Needs triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
folder description not shows when using folder tree to navigateForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertabug feature: folder description
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 58/100
nextcloud/text#9334 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Deleting a file from the editor's own "⋯" menu fails with 423 Locked: Text's own lock blocks itApertabug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 56/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nextcloud/text
Issue simili
-
Add: OnTime Sports SD LogoApertacheck:failed logos:add
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
iptv-org/database#38874 · 1 commento ·
I maintainer di solito rispondono entro 4 giorni
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
githubnext/gh-aw-workshop#5193 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
agent/quality stage/040-covered-by-pr testing
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
cncf/endusers#2031 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno