You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/19 11:22:37 UTC

[GitHub] [airflow] ashb commented on a diff in pull request #25799: Add Airflow specific warning classes

ashb commented on code in PR #25799:
URL: https://github.com/apache/airflow/pull/25799#discussion_r950085661


##########
airflow/exceptions.py:
##########
@@ -338,3 +338,17 @@ class TaskDeferralError(AirflowException):
 
 class PodReconciliationError(AirflowException):
     """Raised when an error is encountered while trying to merge pod configs."""
+
+
+class RemoveInAirflow3DeprecationWarning(DeprecationWarning):

Review Comment:
   https://docs.python.org/3/library/warnings.html
   
   
   
   DeprecationWarning | Base category for warnings about deprecated features when those warnings are intended for other Python developers (ignored by default, unless triggered by code in __main__).
   -- | --
   FutureWarning | Base category for warnings about deprecated features when those warnings are intended for end users of applications that are written in Python.
   -- | --
   PendingDeprecationWarning | Base category for warnings about features that will be deprecated in the future (ignored by default).
   
   Should we make this baseclass be FutureWarning?



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org