potiuk commented on PR #43243: URL: https://github.com/apache/airflow/pull/43243#issuecomment-2441409245
One problem with generating the unique hash with other fields, is that it will be even **more** difficult to add new fields, so probably getting rid of session.merge() everywhere and replacing it with our own way of merging objects and database entries by retrieving the unique id based on the unique fields from the DB before merging is likely a long-term better approach. -- 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]
