On Wed, Jul 8, 2026, at 02:19, Arne Roland wrote: > Hi, > > the attached patch set is my attempt to break it up in a granular way. > I will think about this further, but attached is my current patch, > since it seems much easier to read. > > The difference to before is that the base commit is further broken up > into a series of it's own: > > 1. Only prove FOR KEY joins between two base tables > 2. Mechanics for advanced error messages > 3. Derived tables > 4. GROUP BY and DISTINCT > 5. FILTER (WHERE ...) > 6. Tracking of referenced and referencing filter clauses > The later commits have only minor comment adjustments. The series > should be easier to read now, since the individual steps with very > different concerns are added one at a time. Please let me know, what > you think.
Nice. I've looked at all ten patches, and I like how you've broken it up. I note that src/backend/parser/parse_key_join.c is now "just" 1487 loc in 0001. /Joel
