[GitHub] [airflow] yesemsanthoshkumar commented on pull request #10343: [WIP] Add extra links for google dataproc

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

yesemsanthoshkumar commented on pull request #10343:
URL: https://github.com/apache/airflow/pull/10343#issuecomment-674410120


   @mik-laj 
   I've added extra link for dataproc jobs.
   
   However, I've the following questions.
   
   1. While running the pre-commit hooks, I see both pylint and fake8 running checks. I've marked both of them to disable line too long error for some lines inside the files. Why do we have both of them?
   2. I'm directing the link to the cluster if the job hasn't run and not generated any JobId. Is this approach valid? Do you want me to change this behaviour?
   3. I coudn't find the cluster_name argument in [DataprocSubmitJobOperator](https://airflow.readthedocs.io/en/latest/_api/airflow/providers/google/cloud/operators/dataproc/index.html#airflow.providers.google.cloud.operators.dataproc.DataprocSubmitJobOperator) for which I've added the links. Shoud I add the links to [DataprocJobBaseOperator](https://airflow.readthedocs.io/en/latest/_api/airflow/providers/google/cloud/operators/dataproc/index.html#airflow.providers.google.cloud.operators.dataproc.DataprocJobBaseOperator)? Or Should I add on both?


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