Entry point for DefaultImpl subclasses? (or, just use plugins?)
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Review Alembic's existing plugin ecosystem and the linked pull request #1875, then compare that approach with the proposed alembic.implementations entry point. Done means the project has a clear decision on registration order and whether the existing plugin pattern should be documented or a new entry point supported.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Discussed in https://github.com/sqlalchemy/alembic/discussions/1872
Originally posted by dwreeves September 30, 2026
Hello,
I could not find any discussion on this elsewhere.
I'm wondering if it would make sense for Alembic to support entry-points for Defaultmpl subclasses.
Right now, it looks like the pattern some third-party providers use (e.g. sqlalchemy-redshift) is to load the impl as a side-effect of importing the SqlAlchemy dialect module entry-point.
There are two advantages of an entry-point. The first would be not needing to import Alembic or register anything during the normal runtime of the application. Not usually a big deal to have Alembic in the sys modules, but could be nice-to-have for custom dialects.
The second is a little more subtle, but the entry-point invocation could control the order of registration of dialects, so that the env.py's dialect is guaranteed to resolve last.
The second is actually relevant to a PR I have open for snowflake-sqlalchemy: right now there is no guarantee of whether the library provided SnowflakeImpl will resolve before or after a user's custom implementation; it's totally dependent on the env.py's imports. If entry-points are respected however, then Alembic can always resolve entry-points before env.py runs, guaranteeing user-generated DefaultImpls are resolved last. https://github.com/snowflakedb/snowflake-sqlalchemy/issues/758
So e.g. I have a SQLAlchemy driver for MyCoolSqlDatabase, I could structure like this:
pyproject.toml
src/
mycoolsqldatabase_sqlalchemy/
__init__.py
dialect.py
alembic.py
# pyproject.toml
[project.entry-points."sqlalchemy.dialects"]
mycoolsqldatabase = "mycoolsqldatabase_sqlalchemy:dialect"
[project.entry-points."alembic.implementations"]
mycoolsqldatabase = "mycoolsqldatabase_sqlalchemy:alembic" # alternatively mycoolsqldatabase_sqlalchemy.alembic:MyCoolSqlDatabaseImpl
An argument against this--
I am aware that Alembic uses entry-points for plugins.
Perhaps the pattern you might suggest is to create a plugin with an empty setup() that registers the Alembic impl as a side-effect? E.g.
# plugin.py
from alembic.ddl.impl import DefaultImpl
class MyCoolSqlDatabaseImpl(DefaultImpl):
__dialect__ = "my_cool_sql_database"
def setup(plugin):
pass
If so, it may make sense to show that as an example in the documentation. If that is guaranteed to resolve before env.py, then that could also work as a path for the aforementioned PR.
I think the upside to using the existing plugin ecosystem is that it reduces the API surface on Alembic's end, but the downside is this is very cheeky and not intuitive, especially without a documented example.
Lastly, thanks for the awesome ecosystem, zzzeeek.
- Lenguaje dominante
- Python
- Estrellas
- 4.5k
- Forks
- 391
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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.
Más de sqlalchemy/alembic
-
on_version_apply is typed as a positional callable, but alembic calls it with keyword argumentsQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
sqlalchemy/alembic#1883 · 1 comentario ·
-
Revision message is rendered unescaped into the migration docstring; triple quotes and backslashes break the generated scriptQuizá libre de nuevo @ApparentlyTejas la tomó hace 21 días y no hay ningún pull request abierto. Abiertobug command interface
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
sqlalchemy/alembic#1879 · 3 comentarios ·
-
bug pep 484
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
sqlalchemy/alembic#1341 · 2 comentarios ·
-
autogenerate - detection bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
sqlalchemy/alembic#1121 · 3 comentarios ·
-
Identity and Computed comparison skipped when compare_server_default is not set, as of 1.18.0Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertoautogenerate - defaults bug regression
Dificultad 3/5 1-2 días Aptitud para principiantes 50/100
sqlalchemy/alembic#1877 · 1 comentario ·
Todos los issues de sqlalchemy/alembic
Issues similares
-
SslLibrary crashes with unhandled OSError: port/proto not found on non-standard SSL portsPosiblemente ocupada @himanchalkaushale la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
OWASP/Nettacker#1884 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
A-connection-core C-enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
aws-samples/sample-per-model-bedrock#869 ·
Los mantenedores suelen responder en 1 día
-
Spec: say that included_antigen_sources gates LENS labels; do not merge it into antigen kindsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
fzn0x/watchtower#17 ·