Consider Removing New Line Characters when Comparing Computed Constructs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
Línea de trabajo
Empieza en alembic/autogenerate/compare.py, alrededor de las líneas 1047-1055, donde se sanea el contenido de las cláusulas calculadas de la base de datos y de la configuración local antes de compararlas. Reproduce el problema con una Computed Column multilínea en PostgreSQL y verifica después que las cláusulas equivalentes con saltos de línea colocados de forma diferente ya no produzcan la computed-default warning.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I'm working with a Computed Column construct, which is really long and therefore contains line breaks for readability.
When this construct gets imported into the underlying database (Postgres in my case), the construct gets reformatted with line breaks inserted in different places.
This creates a small side-effect, where alembic now thinks that the original computed clause provided by the migration file is different to the one in the database, and therefore emits a warning.
/usr/local/lib/python3.10/site-packages/alembic/autogenerate/compare.py:1090:
UserWarning: Computed default on *** cannot be modified
util.warn("Computed default on %s.%s cannot be modified" % (tname, cname))
After inspecting the code, I found this method, which appears to sanitize the database computed clause as well as the local one:
I feel that we should include line breaks in the list of tokens that should be removed before a comparison takes place, but happy to hear what thoughts you have on this.
Thanks!
- 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 ·
-
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 ·
-
Entry point for DefaultImpl subclasses? (or, just use plugins?)Posiblemente ocupada @dwreeves la tomó hace 40 días. Abiertocode review in progress migration environment use case
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
sqlalchemy/alembic#1874 · 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 ·