potiuk commented on pull request #21020:
URL: https://github.com/apache/airflow/pull/21020#issuecomment-1020488924


   Right - rebased and fixed last two errors: @uranusjr @ashb - I fixed two 
small thing (not even errors but inconsistencies and MyPy complaints in the new 
`Operator` Interface - I am not sure if adding "owner" to interface is a good 
idea - I think think we do not need to add dag though (as get_dag() shoudl 
return it anyway):
   
   The errors were:
   
   ```
   airflow/models/base.py:67: error: "Operator" has no attribute "dag" 
   [attr-defined]
                   return self.dag.dag_id
                          ^
   airflow/models/base.py:68: error: "Operator" has no attribute "owner" 
   [attr-defined]
               return f"adhoc_{self.owner}"
   
   
   ```
   


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