GitHub user hy144328 added a comment to the discussion: Roadmap to SQLAlchemy 2

> Merged #41917! Thank you! #41915 also merged.
> 
> I simply rebased #41914 for now, but I'm happy to close it out if that's 
> messing up your implementation plan.

Ah, sorry. I only check #41914 for stuff relevant to #41917 .
I might have caused duplicate work by creating #41981 and some small PRs.
In general, I am open to merging in your #41914 , and then see if my smaller 
PRs pick up any additional stuff.
Since four eyes see more than two eyes, and some things were definitely caught 
by unit tests.

GitHub link: 
https://github.com/apache/superset/discussions/40273#discussioncomment-17628021

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: 
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to