mik-laj commented on a change in pull request #16931:
URL: https://github.com/apache/airflow/pull/16931#discussion_r702481903
##########
File path: airflow/timetables/base.py
##########
@@ -94,6 +95,15 @@ def logical_date(self) -> DateTime:
class Timetable(Protocol):
"""Protocol that all Timetable classes are expected to implement."""
+ @cached_property
+ def interval_description(self) -> Optional[str]:
+ """Override to describe the interval.
+
+ For cron ``'30 21 * * 5'``, description could be like ``'At 09:30 PM,
only on Friday'``.
+ This is used in the web UI.
+ """
+ return None
Review comment:
```suggestion
interval_description: Optional[str] = None
"""Override to describe the interval.
For cron ``'30 21 * * 5'``, description could be like ``'At 09:30 PM,
only on Friday'``.
This is used in the web UI.
"""
```
This is a protocol, so I don't reason see why we should use cached_propertty.
--
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]
