Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

register_vector leaks dumper classes on every call

Offen
  • Stimmt etwas nicht?
#168 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
55/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
postgresql, python
Bereich
backend, databases

Rechercherichtung

Start with register_vector_info and the dumper classes in pgvector/psycopg/vector.py, then compare their registration with psycopg’s hstore.py example linked in the issue. Use the supplied reproduction with psycopg’s C extension to check class counts before and after repeated calls; done means the dumper classes are no longer retained after contexts are deleted.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Hi!
Investigating a memory leak in our API, we found out that register_vector (and register_vector_async) creates 8 classes on every call that are never garbage collected if psycopg's C extension is in use, even after the connection is closed.

This is because these classes are stored by psycopg in a dict shared by the whole python process, and never cleared.

pgvector python 0.4.2 (but the code has not been updated on master) and psycopg 3.3.2

To reproduce:

import gc
from types import SimpleNamespace

import psycopg
from pgvector.psycopg.vector import (
    VectorBinaryDumper,
    VectorDumper,
    register_vector_info,
)
from psycopg.adapt import AdaptersMap
from psycopg.types import TypeInfo


def count() -> int:
    gc.collect()
    return sum(
        1
        for base in (VectorDumper, VectorBinaryDumper)
        for c in base.__subclasses__()
        if c.__name__ == ""
    )


print(psycopg.__version__, psycopg.pq.__impl__)
before = count()
for _ in range(1000):
    ctx = SimpleNamespace(adapters=AdaptersMap(psycopg.adapters))
    register_vector_info(ctx, TypeInfo("vector", 16385, 16386))
    del ctx
print("leaked dumper classes:", count() - before)  # expected 0, prints 2000

Caching these classes as done by psycopg could fix the issue : https://github.com/psycopg/psycopg/blob/99356800e90aa0e7b604f652d714ef0f33422b47/psycopg/psycopg/types/hstore.py#L225

Vorherrschende Sprache
Python
Sterne
1.5k
Forks
100
Ø Merge
13 Min.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

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

Mehr aus pgvector/pgvector-python

Alle Issues in pgvector/pgvector-python

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

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