ashb commented on a change in pull request #15046:
URL: https://github.com/apache/airflow/pull/15046#discussion_r603178006
##########
File path: airflow/configuration.py
##########
@@ -253,6 +253,18 @@ def _validate_config_dependencies(self):
+ ", ".join(start_method_options)
)
+ if self.has_option("scheduler", "file_parsing_sort_mode"):
+ list_mode = self.get("scheduler", "file_parsing_sort_mode")
+ file_parser_modes = {"modified_time", "random_seeded_by_host",
"alphabetical"}
+
+ if list_mode not in file_parser_modes:
+ raise AirflowConfigException(
+ "`[scheduler] file_parsing_sort_mode` should not be "
+ + list_mode
Review comment:
Use an f-string too maybe for this whole lote?
--
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]
