MagicStack / MagicStack/asyncpg
Temporary tables still visible after releasing connection to pool
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 8.1k
- Forks
- 468
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
It seems like releasing a connection back to the pool doesn't entirely reset its state in a way that makes it brand new for the next user. Is it expected that temporary tables are still visible? Temporary tables in PostgreSQL are dropped after the session exits but wondering if there is a missing incantation in Connection.reset() that will allow it to be reused.
import asyncio
import asyncpg
async def main():
pool = await asyncpg.create_pool(
...
min_size=1,
max_size=1,
)
async with pool.acquire() as conn:
await conn.execute("CREATE TEMPORARY TABLE foo(id INT PRIMARY KEY)")
async with pool.acquire() as conn:
await conn.execute("CREATE TEMPORARY TABLE foo(id INT PRIMARY KEY)") # DuplicateTableException
asyncio.run(main())
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza ejecutando la reproducción de asyncio proporcionada con un pool de una sola conexión y, a continuación, inspecciona Connection.reset() y la ruta de adquisición/liberación del pool. Determina si liberar una conexión debería eliminar la tabla temporal y haz que el comportamiento coincida con las expectativas documentadas de restablecimiento del pool, usando la reproducción como comprobación de finalización.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- postgresql, python
- Área
- databases
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100