Skip to content

Conversation

@mtoy-googly-moogly
Copy link
Collaborator

@mtoy-googly-moogly mtoy-googly-moogly commented Sep 26, 2025

ok there are four pieces to this

  1. mark field usage from an on expression as coming from an on expression
  2. when expanding field usage, propogate the on expression to all fields needed to satisfy the on expression
  3. when building the active join list, look for joins with on expressions which go through another join and mark that
  4. when generating joins, for marked joins, move the on expression to the filters and make the on expression be 1=1

the approach was suggested by lloyd
the hard parts of this were written by and and claude

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants