Commit 51c0f63

committed

Improve handling of pathtargets in planner.c.

Refactor so that the internal APIs in planner.c deal in PathTargets not targetlists, and establish a more regular structure for deriving the targets needed for successive steps. There is more that could be done here; calculating the eval costs of each successive target independently is both inefficient and wrong in detail, since we won't actually recompute values available from the input node's tlist. But it's no worse than what happened before the pathification rewrite. In any case this seems like a good starting point for considering how to handle Konstantin Knizhnik's function-evaluation-postponement patch.

1 parent 2f1f443 commit 51c0f63Copy full SHA for 51c0f63

4 files changed

+190

-118

lines changed

src
- backend/optimizer
  - plan
    - planner.c
  - util
    - clauses.c
    - tlist.c
- include/optimizer
  - tlist.h

4 files changed

+190

-118

lines changed

Comments

(0)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 51c0f63

4 files changed

4 files changed

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier! Saves Data!

File tree

4 files changed

4 files changed

0 commit comments

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier! Saves Data!