potiuk commented on PR #72243: URL: https://github.com/apache/airflow/pull/72243#issuecomment-5462587052
Have you thought about side-effect of it? What are they? Do ypu (not your LLM) understand what your are doing here?have How it can happen that serialized dag is missing? Is it maybye a sign that something else is wrong - and you are just masking a problem? Note - I have years of experience in Airflow but if I were to touch this code - I would think100 times and talk to someone over slack explaining how I reproduced . Did you actually experience and reproduce it on running Airlfow instance? Do you have some proof of that? Looking at the patterns of your contribution - you contribute like a shotgun - wherever your LLM thinks there is an issue, but you do not have deeper undersrtanding. i am provisionally closing that - the "critical" parts of the code shoudl not be contributed to significantly without either experience or significant proof that you have run it locally, reproduced the issue and solved it and that you thought and reasoned about the consequences. Thare are other areas where you can contribute smaller things in Airflow - if all you do is put your LLLm on it. -- 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]
