airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [airflow] kaxil commented on a change in pull request #6302: [AIRFLOW-5636] Allow adding or overriding existing Operator Links for…
Date Tue, 15 Oct 2019 12:37:31 GMT
kaxil commented on a change in pull request #6302: [AIRFLOW-5636] Allow adding or overriding
existing Operator Links for…
URL: https://github.com/apache/airflow/pull/6302#discussion_r334923758
 
 

 ##########
 File path: airflow/models/baseoperator.py
 ##########
 @@ -566,7 +566,20 @@ def priority_weight_total(self):
     @cached_property
     def operator_extra_link_dict(self):
         """Returns dictionary of all extra links for the operator"""
-        return {link.name: link for link in self.operator_extra_links}
+        from airflow.plugins_manager import operator_extra_links
+
+        op_extra_links_from_plugin = {}
+        for ope in operator_extra_links:
+            if ope.operators and self.__class__ in ope.operators:
 
 Review comment:
   Agree

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message