Taragolis commented on code in PR #36876:
URL: https://github.com/apache/airflow/pull/36876#discussion_r1457894304


##########
airflow/providers/google/cloud/links/dataproc.py:
##########
@@ -71,13 +70,16 @@ def __getattr__(name: str) -> Any:
 )
 
 
-@attr.s(auto_attribs=True)
+@deprecated(
+    reason="This DataprocLink is deprecated.",
+    category=AirflowProviderDeprecationWarning,
+)

Review Comment:
   Would be surprised if it not cause failure in static checks.
   This class uses in some operators in top level, so this is why it pending to 
deprecate, without actual deprecation warnings



##########
airflow/deprecation.py:
##########


Review Comment:
   Seems like this module do not use anywhere, and make changes into the core, 
so better add it by the separate PR



-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to