buxizhizhoum commented on a change in pull request #16584:
URL: https://github.com/apache/airflow/pull/16584#discussion_r657279141
##########
File path: airflow/models/dag.py
##########
@@ -694,22 +701,27 @@ def has_dag_runs(self, session=None,
include_externally_triggered=True) -> bool:
@property
def dag_id(self) -> str:
+ """Get the dag_id"""
return self._dag_id
@dag_id.setter
def dag_id(self, value: str) -> None:
+ """Set dag_id"""
self._dag_id = value
@property
def full_filepath(self) -> str:
+ """Dag location"""
return self._full_filepath
@full_filepath.setter
def full_filepath(self, value) -> None:
+ """update full filepath"""
self._full_filepath = value
@property
def concurrency(self) -> int:
+ """Max active tasks limit of a dag"""
Review comment:
I have removed less helpful doc strings including this one, thanks for
your advice:)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
