uranusjr commented on code in PR #39109:
URL: https://github.com/apache/airflow/pull/39109#discussion_r1571823694
##########
airflow/models/dagbag.py:
##########
@@ -67,13 +68,23 @@
class FileLoadStat(NamedTuple):
- """Information about single file."""
+ """
+ Information about single file.
+
+ :param file: Loaded file.
+ :param duration: Time spent on process file.
+ :param dag_num: Total number of DAGs loaded in this file.
+ :param task_num: Total number of Tasks loaded in this file.
+ :param dags: DAGs names loaded in this file.
+ :param warnings: Total warnings captured during the process file.
Review Comment:
```suggestion
:param warnings: Total number of warnings captured during the process
file.
```
A name such as `warning_num` might be better as well?
##########
airflow/models/dagbag.py:
##########
@@ -67,13 +68,23 @@
class FileLoadStat(NamedTuple):
- """Information about single file."""
+ """
+ Information about single file.
+
+ :param file: Loaded file.
+ :param duration: Time spent on process file.
+ :param dag_num: Total number of DAGs loaded in this file.
+ :param task_num: Total number of Tasks loaded in this file.
+ :param dags: DAGs names loaded in this file.
+ :param warnings: Total warnings captured during the process file.
Review Comment:
```suggestion
:param warnings: Total number of warnings captured from processing this
file.
```
A name such as `warning_num` might be better as well?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
