Skip to content

Commit

Permalink
Fix update query and increase simultaneous tasks of the `add_license_…
Browse files Browse the repository at this point in the history
…url` DAG (#4661)
  • Loading branch information
krysal authored Jul 26, 2024
1 parent 4aec507 commit 3e3799a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions catalog/dags/maintenance/add_license_url.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ def get_license_conf(license_info) -> dict:
f"AND meta_data->>'license_url' IS NULL"
),
# Merge existing metadata with the new license_url
"update_query": f"SET meta_data = ({Json(license_url_dict)}::jsonb || meta_data), updated_on = now()",
"update_query": f"SET updated_on = NOW(), meta_data = ({Json(license_url_dict)}::jsonb || meta_data)",
"update_timeout": 259200, # 3 days in seconds
"dry_run": False,
"resume_update": False,
Expand Down Expand Up @@ -194,7 +194,7 @@ def add_license_url():
trigger_dag_id=BATCHED_UPDATE_DAG_ID,
wait_for_completion=True,
execution_timeout=timedelta(hours=5),
max_active_tis_per_dag=1,
max_active_tis_per_dag=3,
map_index_template="""{{ task.conf['query_id'] }}""",
retries=0,
).expand(conf=get_confs(licenses, batch_size="{{ params.batch_size }}"))
Expand Down

0 comments on commit 3e3799a

Please sign in to comment.