You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/17 08:05:21 UTC

[GitHub] [airflow] suiting-young commented on pull request #14821: `task_type` should be respected as it is supposed to be overridden.

suiting-young commented on pull request #14821:
URL: https://github.com/apache/airflow/pull/14821#issuecomment-800882405


   > was not intended to have a Human Readable name.
   
   My use case isn't to place arbitrary sentences there, but create some *fake*-sub-classes (that my users will treat them as *real* class names).
   
   In 21a91c4cd060a510f8835db0346f725c9fe45ca4 (*2nd commit*) I extended `BaseOperatorLink.operators` to accept `str`.
   So that `task_type`-customized classes can have their own links.
   
   **This won't affect existing code** because
   - No operator overrode `task_type` in official source tree.
   - Custom installation has no reason (*and actually, no effect*) to override `task_type` without 328a43afcba143314f19831a7586db91bb3f9678 (*1st commit*).


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