Re: Removing unneeded self joins
Robert Haas <robertmhaas@gmail.com>
Commits
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Remove GUC_NOT_IN_SAMPLE from enable_self_join_elimination
- 717d0e8dd945 18.0 landed
-
Put enable_self_join_elimination into postgresql.conf.sample
- c2d329260cd8 18.0 landed
-
Get rid of ojrelid local variable in remove_rel_from_query()
- e167191dc146 18.0 landed
-
Implement Self-Join Elimination
- fc069a3a6319 18.0 cited
-
Revert: Remove useless self-joins
- d1d286d83c0e 17.0 landed
-
Replace lateral references to removed rels in subqueries
- 466979ef031a 17.0 landed
-
Replace relids in lateral subquery parse tree during SJE
- 489072ab7a9e 17.0 landed
-
Forbid SJE with result relation
- 8c441c082797 17.0 landed
-
Fix misuse of RelOptInfo.unique_for_rels cache by SJE
- 30b4955a4668 17.0 landed
-
Replace the relid in some missing fields during SJE
- a7928a57b9f0 17.0 landed
-
Revert 56-bit relfilenode change and follow-up commits.
- a448e49bcbe4 16.0 cited
-
Stabilize timetz test across DST transitions.
- 4a071afbd056 14.0 cited
-
Speed up finding EquivalenceClasses for a given set of rels
- 3373c7155350 13.0 cited
-
Fix mark-and-restore-skipping test case to not be a self-join.
- 24d08f3c0a1f 12.0 landed
On Wed, May 16, 2018 at 6:13 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote: > David Rowley <david.rowley@2ndquadrant.com> writes: >> On 17 May 2018 at 08:44, Simon Riggs <simon@2ndquadrant.com> wrote: >>> What I was advocating was an approach that varies according to the >>> query cost, so we don't waste time trying to tune the heck out of OLTP >>> queries, but for larger queries we might take a more considered >>> approach. > >> That's tricky. If we do this, it should be done before Path >> generation, so not much is known about the costs in those case. > > Yeah. It'd have to be a very heuristic thing that doesn't account > for much beyond the number of relations in the query, and maybe their > sizes --- although I don't think we even know the latter at the > point where join removal would be desirable. (And note that one of > the desirable benefits of join removal is not having to find out the > sizes of removed rels ... so just swapping that around doesn't appeal.) As I've mentioned before, the problem we're talking about here is also highly relevant to parallel query. We only want to bother generating partial paths for queries that are expensive enough to justify considering parallelism, but we don't know how expensive they are until we finishing planning. The only way I could think of to tackle that problem was to drive it off the relation sizes, but the presence or absence of expensive functions in the target list can wildly change the point at which parallelism potentially becomes useful. So we end up sometimes wasting effort generating partial paths that are totally useless, and at other times failing to generate partial paths that would have been useful. (Really, I'd like to generate a lot more partial paths than we do, trying out various numbers of workers, but that would just make the existing problem worse.) I have wondered about doing a preliminary pass over the tree where we try to make a crude estimate of the amount of effort involved, and then planning for real with that number in hand. But it seems like there's so little information you could get at that early stage that it would be hard to decide anything useful on that basis. You've got to at least have relation sizes, and really you need some estimate of the result cardinality as well. It seems like we currently can't figure out cardinality without also computing paths, and I've wondered if we could split those into two separate phases. But every time I think about trying to do that I realize that my pain tolerance isn't that high. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company