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:
[email protected]


Reply via email to