mikaelfr commented on pull request #9204: URL: https://github.com/apache/airflow/pull/9204#issuecomment-648749495
> Please provide screenshots of the new UI functionality. I edited my original comment to include some. > I'm not sure storing this against TI is actually the right thing to do -- the task tag is a property of the Task, not the TaskInstance. What about instead if we just looked at the tags property of `dag.tasks`. > > i.e. something like `tasks = [task for dag.tasks if task.has_tag(val)]` > > (With the planned "DAG versioning" where we store a different version of the serialized dag model when it changes any concern about this being out of date with latest definition goes away) I'm not sure how difficult it would be to extract tasks from serialized dags in FAB TaskInstanceModelView and use those tasks to somehow filter task instances by tags. What I can gather from my limited experience with FAB, it seems that it's not very kind to doing things in ways differing from the norm. ---------------------------------------------------------------- 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]
