apache/airflow

Don't have session.commit() when @provide_session decorator is used

Open

#12.818 aberto em 4 de dez. de 2020

Ver no GitHub
 (10 comments) (0 reactions) (1 assignee)Python (16.781 forks)batch import
area:MetaDBcontributors-workshopgood first issuekind:taskpriority:medium

Métricas do repositório

Stars
 (44.809 stars)
Métricas de merge de PR
 (Mesclagem média 7d 18h) (834 fundiu PRs em 30d)

Description

As discussed in https://github.com/apache/airflow/pull/12815#discussion_r536283503 and pointed out at https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#database-session-handling, we should not have session.commit() when @provide_session decorator is used (for principle "Explicit is better than implicit").

This issue exists in files like https://github.com/apache/airflow/blob/2936c13a44369daae9d6099cd1070b3a8cd53443/airflow/models/dag.py (and some more of course)

Guia do colaborador

Direção de pesquisa
Encontre todas as ocorrências de session.commit() em arquivos que usam o decorador @provide session e remova as, seguindo o princípio 'Explícito é melhor que implícito' conforme as diretrizes de contribuição do Airflow.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Manutenção
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80