MagicStack / MagicStack/asyncpg
Multihost bug when working with SqlAlchemy
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 8.1k
- Fork
- 468
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
* **asyncpg version**: 0.24.0
* **PostgreSQL version**: 10
* **Do you use a PostgreSQL SaaS? If so, which? Can you reproduce
the issue with a local PostgreSQL install?**:
* **Python version**: 3.9.6
* **Platform**: Doesn't matter
* **Do you use pgbouncer?**: No
* **Did you install asyncpg with pip?**: Yes
* **If you built asyncpg locally, which version of Cython did you use?**:
* **Can the issue be reproduced under both asyncio and
[uvloop](https://github.com/magicstack/uvloop)?**: doesn't matter
When we use asyncpg via sqlalchemy, we pass a dsn string to sqlalchemy and it parses this string and call asyncpg connect with host, port, username and etc instead of dsn. And it looks like asyncpg can't handle multiple hosts, separated by commas, passed in host param.
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.
Direzione di ricerca
Inizia tracciando il modo in cui SQLAlchemy analizza il DSN e chiama asyncpg con il parametro host, quindi riproduci il caso con più host PostgreSQL separati da virgole usando le versioni indicate. Il lavoro è completato quando il comportamento della connessione multihost funziona nel passaggio del valore host analizzato da SQLAlchemy a asyncpg.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- postgresql, python, sqlalchemy
- Ambito
- databases
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 32/100