MerginMaps / MerginMaps/python-api-client

Cleanup of failed download may misbehave

Offen
#157 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug
Vorherrschende Sprache
Python
Sterne
23
Forks
12
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
7

Beschreibung

Sometimes if a download fails, the subsequent cleanup of the project directory may fail as well: https://github.com/MerginMaps/qgis-mergin-plugin/issues/442#issuecomment-1212909680
While it looks like some other app is using the downloaded files, it is possible that we are doing the cleanup too early - e.g. in download_project_is_running() we do the cleanup while some other futures may be still active: we should wait until all jobs are done before doing any cleanup.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne bei download_project_is_running() und verfolge, wie die Bereinigung des Projektverzeichnisses mit den anderen aktiven Futures zusammenhängt. Verwende das verlinkte Mergin Maps-Issue für das gemeldete Fehlerszenario. Als erledigt gilt die Aufgabe, wenn die Bereinigung wartet, bis alle Download-Aufträge beendet sind, und nicht mehr fehlschlägt, wenn ein Download fehlschlägt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
api
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.