[GitHub] [airflow] potiuk commented on issue #25431: Operator deepcopy does not honor `shallow_copy_attrs` if value also referenced in `__init_kwargs`

[GitBox <gi...@apache.org>]

potiuk commented on issue #25431:
URL: https://github.com/apache/airflow/issues/25431#issuecomment-1203072620

   I think if you want to propose something, it's better to open PR and discuss it there adding your explanation over the code. This will make it far more productive discussion than trying to wrap the head around copy & pasted code from various places. This will take a long time for anyone looking at it to spend their energy on, and making a draft PR with what you proposed to do is far better IMHO. I have now big knowledge about this part, but I kinda dread having to take a look and try to understand what you want to do and why, becaus of all the copy &pasted code. 
   
   Just sayin;


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org