GitHub user rusackas added a comment to the discussion: Roadmap to SQLAlchemy 2
Chipped away at three more of the cascade_backrefs cases: Query (#42210), SavedQuery (#42212), and SqlaTable (#42213). Turns out the callers were already adding everything to the session explicitly, so each one is just `cascade_backrefs=False` on the backref plus flipping the pytest.ini line on. The only production code leaning on the reverse cascade was an expunge in `raise_for_access`, now guarded. SqlMetric, TableColumn, and TaggedObject look like the same shape if anyone wants to grab them... the "User" one is trickier since `Role.user` lives in Flask-AppBuilder. @hy144328 holler if any of this collides with what you have in flight. GitHub link: https://github.com/apache/superset/discussions/40273#discussioncomment-17685626 ---- 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]
