MerginProject may not get cleaned up properly

Offen

#149 erstellt am 18.10.2022 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: api

Rechercherichtung

Beginne damit, die Erstellung von MerginProjects pygeodiff GeoDiff und die Einrichtung des Logger-Callbacks zu lokalisieren, und vergleiche dann den Cleanup-Fix in mergin-db-sync PR 79 mit dem zugehörigen geodiff issue 190. Als erledigt gilt die Aufgabe, wenn die wiederholte Erstellung von MerginProject die zyklischen Referenzen nicht mehr beibehält und keine Ressourcen anwachsen lässt, wobei die Bereinigung mit der Reproduktionsschleife überprüft wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

When MerginProject is created, it creates geodiff object and assigns an internal logger to geodiff. This creates a cyclic reference (pygeodiff refs MerginProject, and MerginProject refs pygeodiff) that may keep resources held, which may be a problem if MerginProject object is created many times. See also fix for this in db-sync: https://github.com/MerginMaps/mergin-db-sync/pull/79

Also somehow related: https://github.com/MerginMaps/geodiff/issues/190

Code snippet to replicate the cleanup issue:

import pygeodiff

class MerginProjectX:

    def __init__(self, directory):

        def _logger_callback(level, text_bytes):
            self.x()   # not a real function, but refs this object...

        self.geodiff = pygeodiff.GeoDiff()
        self.geodiff.set_logger_callback(_logger_callback)

# this will keep consuming more and more resources
for i in range(50000):
    print(i)
    mp = MerginProjectX("x")
    #mp.geodiff = None    # this helps to break ref cycle
    time.sleep(0.005)

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus MerginMaps/python-api-client

Vacuuming Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

MerginMaps/python-api-client#320 · Erstellt am 09.9.2026
Create unified api interface for api integration Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 30/100

MerginMaps/python-api-client#270 · Erstellt am 09.9.2025
Migration tool between servers Offen

enhancement

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

MerginMaps/python-api-client#257 · 3 Kommentare · Erstellt am 06.8.2025
Migrate per-project logs to global logs Offen

enhancement

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

MerginMaps/python-api-client#254 · 1 Kommentar · Erstellt am 24.7.2025
Impose geodiff push limits Offen

enhancement

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

MerginMaps/python-api-client#245 · Erstellt am 19.6.2025

Alle Issues in MerginMaps/python-api-client