[GitHub] [airflow] Taragolis commented on pull request #24294: Refactoring EmrClusterLink and add it for other AWS EMR Operators

[GitBox <gi...@apache.org>]

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

   >I wonder if we can figure out a way to work the operator_extra_links assignment into the AwsBaseOperator that you are working on as well, so we don't assign it each time. 
   
   I think it is hardly possible because each operator could required each set of Links and not all of them related to Hook
   
   Example:
   - For EMR at that moment only link to cluster could be provided because there is no direct link to steps exists.
   - For AWS Batch it could be set of links: Batch Job Detail, Latest Attempt logs in CloudWatch, Batch Job Definition, Batch Queue
   - For Glue Job it could be set of links: Glue Job Detail (Glue ETL Studio), Glue Job Detail (Legacy), Glue Job, Filter to CloudWatch prefixes 
   


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