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_r334921663
 
 

 ##########
 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:
 
 Review comment:
   no, it is intentional.
   
   `operator_extra_links` is from plugins
   
   `self.operator_extra_links` is from the operator itself

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


With regards,
Apache Git Services

Reply via email to