Skip to content

Commit f3c22a5

Browse files
dirraoromsharon98
authored andcommitted
Airflow 3: store_serialized_dags deprecated parameter usage removal (apache#40931)
* store_serialized_dags parameter usage removal * news fragment added
1 parent a64e3b7 commit f3c22a5

File tree

2 files changed

+1
-23
lines changed

2 files changed

+1
-23
lines changed

airflow/models/dagbag.py

-23
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
import sys
2626
import textwrap
2727
import traceback
28-
import warnings
2928
import zipfile
3029
from datetime import datetime, timedelta
3130
from pathlib import Path
@@ -47,7 +46,6 @@
4746
AirflowDagCycleException,
4847
AirflowDagDuplicatedIdException,
4948
AirflowException,
50-
RemovedInAirflow3Warning,
5149
)
5250
from airflow.listeners.listener import get_listener_manager
5351
from airflow.models.base import Base
@@ -113,7 +111,6 @@ class DagBag(LoggingMixin):
113111
to filter python modules to scan for dags.
114112
:param read_dags_from_db: Read DAGs from DB if ``True`` is passed.
115113
If ``False`` DAGs are read from python files.
116-
:param store_serialized_dags: deprecated parameter, same effect as `read_dags_from_db`
117114
:param load_op_links: Should the extra operator link be loaded via plugins when
118115
de-serializing the DAG? This flag is set to False in Scheduler so that Extra Operator links
119116
are not loaded to not run User code in Scheduler.
@@ -126,7 +123,6 @@ def __init__(
126123
include_examples: bool | ArgNotSet = NOTSET,
127124
safe_mode: bool | ArgNotSet = NOTSET,
128125
read_dags_from_db: bool = False,
129-
store_serialized_dags: bool | None = None,
130126
load_op_links: bool = True,
131127
collect_dags: bool = True,
132128
):
@@ -143,15 +139,6 @@ def __init__(
143139
safe_mode if isinstance(safe_mode, bool) else conf.getboolean("core", "DAG_DISCOVERY_SAFE_MODE")
144140
)
145141

146-
if store_serialized_dags:
147-
warnings.warn(
148-
"The store_serialized_dags parameter has been deprecated. "
149-
"You should pass the read_dags_from_db parameter.",
150-
RemovedInAirflow3Warning,
151-
stacklevel=2,
152-
)
153-
read_dags_from_db = store_serialized_dags
154-
155142
dag_folder = dag_folder or settings.DAGS_FOLDER
156143
self.dag_folder = dag_folder
157144
self.dags: dict[str, DAG] = {}
@@ -182,16 +169,6 @@ def size(self) -> int:
182169
""":return: the amount of dags contained in this dagbag"""
183170
return len(self.dags)
184171

185-
@property
186-
def store_serialized_dags(self) -> bool:
187-
"""Whether to read dags from DB."""
188-
warnings.warn(
189-
"The store_serialized_dags property has been deprecated. Use read_dags_from_db instead.",
190-
RemovedInAirflow3Warning,
191-
stacklevel=2,
192-
)
193-
return self.read_dags_from_db
194-
195172
@property
196173
def dag_ids(self) -> list[str]:
197174
"""

newsfragments/40931.significant.rst

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Removed dagbag deprecated ``store_serialized_dags`` parameter. Please use ``read_dags_from_db`` parameter.

0 commit comments

Comments
 (0)