Skip to content

Update airflow_db_cleanup.py to also clean sessions table #10775

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rafalbiegacz merged 2 commits into from
Oct 18, 2023
Merged

Update airflow_db_cleanup.py to also clean sessions table #10775

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
dcfff64
Update airflow_db_cleanup.py to also clean sessions table
apilaskowski Oct 17, 2023
798ec5b
Update airflow_db_cleanup.py
apilaskowski Oct 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
65 changes: 56 additions & 9 deletions composer/workflows/airflow_db_cleanup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,6 @@

import airflow
from airflow import settings
from airflow.jobs.base_job import BaseJob
from airflow.models import (
DAG,
DagModel,
Expand All @@ -69,7 +68,7 @@
from airflow.version import version as airflow_version

import dateutil.parser
from sqlalchemy import and_, func
from sqlalchemy import and_, func, text
from sqlalchemy.exc import ProgrammingError
from sqlalchemy.orm import load_only

Expand Down Expand Up @@ -101,13 +100,6 @@
# List of all the objects that will be deleted. Comment out the DB objects you
# want to skip.
DATABASE_OBJECTS = [
{
"airflow_db_model": BaseJob,
"age_check_column": BaseJob.latest_heartbeat,
"keep_last": False,
"keep_last_filters": None,
"keep_last_group_by": None,
},
{
"airflow_db_model": DagRun,
"age_check_column": DagRun.execution_date,
Expand Down Expand Up @@ -228,6 +220,35 @@
except Exception as e:
logging.error(e)

if AIRFLOW_VERSION < ["2", "6", "0"]:
try:
from airflow.jobs.base_job import BaseJob
DATABASE_OBJECTS.append(
{
"airflow_db_model": BaseJob,
"age_check_column": BaseJob.latest_heartbeat,
"keep_last": False,
"keep_last_filters": None,
"keep_last_group_by": None,
}
)
except Exception as e:
logging.error(e)
else:
try:
from airflow.jobs.job import Job
DATABASE_OBJECTS.append(
{
"airflow_db_model": Job,
"age_check_column": Job.latest_heartbeat,
"keep_last": False,
"keep_last_filters": None,
"keep_last_group_by": None,
}
)
except Exception as e:
logging.error(e)

default_args = {
"owner": DAG_OWNER_NAME,
"depends_on_past": False,
Expand Down Expand Up @@ -440,16 +461,42 @@ def cleanup_function(**context):
session.close()


def cleanup_sessions():
session = settings.Session()

try:
logging.info("Deleting sessions...")
before = len(session.execute(text("SELECT * FROM session WHERE expiry > now()::timestamp(0);")).mappings().all())
session.execute(text("DELETE FROM session WHERE expiry > now()::timestamp(0);"))
after = len(session.execute(text("SELECT * FROM session WHERE expiry > now()::timestamp(0);")).mappings().all())
logging.info("Deleted {} expired sessions.".format(before-after))
except Exception as e:
logging.error(e)

session.commit()
session.close()


def analyze_db():
session = settings.Session()
session.execute("ANALYZE")
session.commit()
session.close()


analyze_op = PythonOperator(
task_id="analyze_query", python_callable=analyze_db, provide_context=True, dag=dag
)

cleanup_session_op = PythonOperator(
task_id="cleanup_sessions",
python_callable=cleanup_sessions,
provide_context=True,
dag=dag
)

cleanup_session_op.set_downstream(analyze_op)

for db_object in DATABASE_OBJECTS:
cleanup_op = PythonOperator(
task_id="cleanup_" + str(db_object["airflow_db_model"].__name__),
Expand Down