#23853 closed Cleanup/optimization (fixed)
Replace JoinInfo namedtuple with Join class
Description ¶
The rationale for this change is that by using a class we can implement all kinds of different joins (LATER JOINS over subquery) if needed for special cases. In addition, this way feels cleaner to me as the join generation logic now happens in a dedicated class instead of "hard-coding" it to SQLCompiler.
Implementation in https://github.com/django/django/pull/3561
Change History (5)
comment:1 by , 10 years ago
Has patch: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 10 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
comment:3 by , 10 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
This should be ready for commit, see PR for review details.