kaxil commented on code in PR #23317:
URL: https://github.com/apache/airflow/pull/23317#discussion_r860764879
##########
airflow/dag_processing/processor.py:
##########
@@ -559,6 +560,28 @@ def update_import_errors(session: Session, dagbag: DagBag)
-> None:
session.commit()
+ @staticmethod
+ def update_configuration_errors(session: Session, dagbag: DagBag) -> None:
Review Comment:
Hmm naming is hard, "configuration" here led me to think this is something
around configs in airflow.cfg especially the name of Model in
`models/dagconfigurationerror.py`
- DagParsingErrors
- AirflowInputError
or we just have a `AirflowUserError` and we can remove `import_error` table
and use this instead where `ImportError` type is replaced a ParsingError?
(Just thinking it out loud -- no strong opinion on any of the above)
--
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]
