melicheradam commented on PR #68441: URL: https://github.com/apache/airflow/pull/68441#issuecomment-4691615642
> Duplicated docs. > > What about multi line output in logs > > Needs a way to disable the parsing I think. > > If you are willing/already have `airflow.sdk` installed in the Venv, then you get this (log parsing) for free without needing to parse anything. I realise that's not for everyone, but it should be a) tested to check it works, and if it does b) mentioned in the docs please. Hi, thanks for the review. The logging config is not needed if you have `airflow.sdk` in the Venv, I will put that to the docs. Also remove the duplicate parts and test with multiline logs - Ill add that to the unit tests, but I expect it would work since it worked for tracebacks and errors. The main motivation behind this is to not have airflow in the venv. With Airflow 3, it takes significantly longer to just import than on previous version (around 8s), so we started migrating away from keeping airflow in the venvs. That is mainly because we use the google provider, which is huge, and having a 8s delay on each task just to import was way too much. Thats when I noticed the logs do not match in that case, and if you have loglevel globally set > INFO, you would not see any task execution logs. -- 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]
