Taragolis commented on PR #24294:
URL: https://github.com/apache/airflow/pull/24294#issuecomment-1152583999

   > Maybe. But also, if BaseOperator created a Link set containing the link to 
the service's main console page
   
   The implementation of Aws Service link it straightforward
   
   ```python
   import attr
   from airflow.providers.amazon.aws.links.base_aws import BaseAwsLink, 
BASE_AWS_CONSOLE_LINK
   
   
   @attr.s(auto_attribs=True)
   class AwsServiceLink(BaseAwsLink):
       name: str
       format_str: str
       key = "aws_service"
   
   AwsServiceLink(
       name="AWS EMR Console Link",
       format_str=BASE_AWS_CONSOLE_LINK + 
"/elasticmapreduce/home?region={region_name}#"
   )
   ```
   
   The is two things which need to be check and solved
   1. Each operator should call `AwsServiceLink.persist` in execute method of 
operator/sensor
   2. Need to check is it fine to change `operator_extra_links` in runtime
   
   > I'm also not sure what this current implementation is going to look like 
when you want to add a second/third/nth link.
   
   All operator links predefined in operator in that moment
   
   In execute stage call `persist` method of link with appropriate keywords, 
which stored in XCom


-- 
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]

Reply via email to