Lee-W commented on PR #41987:
URL: https://github.com/apache/airflow/pull/41987#issuecomment-2328356517

   > I was thinking it’s weird we’re using DatasetModel here… (DatasetManager 
actually converts them back to Dataset for a user hook too)
   > 
   > I was having problems with this function when adding `name` too. Let me 
rewrite this entire function a bit.
   
   I think the main reason is that we need to check the stored dataset models. 
Do you want me to continue working on this one? Or wait for your PR? Without 
it, some edges cases might fail


-- 
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]

Reply via email to