potiuk commented on PR #29051: URL: https://github.com/apache/airflow/pull/29051#issuecomment-1397754758
> Is the watcher supposed to be the "separator" ? Because in that case, disable_job_queue is part of the teardown and apparently executed before the watcher (I sort by end_date, maybe that's wrong ?) Ah... you are right . Watcher is just for marking success/failure.. Bummer. so AIP-52 it is > Yes, I'd love to, but very personal reason not to: the log collector we use keeps the escape codes uninterpreted, so it actually makes the output less readable, but maybe it's something we should fix on our end, idk. I think we have formatter that is conditional and can detect if you are in terminal or not. That would be enough. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
