uranusjr commented on PR #31662: URL: https://github.com/apache/airflow/pull/31662#issuecomment-1573407799
Hmm, yeah adding a new field is probably the simplest approach. Can you find other uses of `TaskInstance.operator` so maybe we can analyse if this field is simply for UI (if so, we can just change it to use `operator_name`), or do other logic depend on it? Adding a field is relatively expensive on TaskInstance so we should avoid that if possible. Either way, I think we should hold this PR as-is and continue looking into `TaskInstance.operator` in a different PR. -- 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]
