apache/airflow

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

Aperta

#12.818 aperta il 4 dic 2020

Vedi su GitHubGuida contributor
 (10 commenti) (0 reazioni) (1 assegnatario)Python (16.781 fork)batch import
area:MetaDBcontributors-workshopgood first issuekind:taskpriority:medium

Metriche repository

Star
 (44.809 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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)

Guida contributor

Direzione di ricerca
Trova tutte le occorrenze di session.commit() nei file che utilizzano il decoratore @provide session e rimuovile, seguendo il principio 'Esplicito è meglio che implicito' come da linee guida per i contributi di Airflow.
Tech stack
python
Dominio
backend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80