diff options
| author | Tom Lane | 2016-06-10 20:20:03 +0000 |
|---|---|---|
| committer | Tom Lane | 2016-06-10 20:20:03 +0000 |
| commit | 3303ea1a327b41d3b406d7be7a5ce2901e561066 (patch) | |
| tree | d3efb62c93fa191a290e101a7766b2e0f58f27dd /src/include/nodes | |
| parent | 2f153ddfdd318b211590dd5585f65f357d85c2de (diff) | |
Remove reltarget_has_non_vars flag.
Commit b12fd41c6 added a "reltarget_has_non_vars" field to RelOptInfo,
but failed to maintain it accurately. Since its only purpose was to skip
calls to has_parallel_hazard() in the simple case where a rel's targetlist
is all Vars, and that call is really pretty cheap in that case anyway, it
seems like this is just a case of premature optimization. Let's drop the
flag and do the calls unconditionally until it's proven that we need more
smarts here.
Diffstat (limited to 'src/include/nodes')
| -rw-r--r-- | src/include/nodes/relation.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/include/nodes/relation.h b/src/include/nodes/relation.h index 7ca1e253b9e..cafd6962298 100644 --- a/src/include/nodes/relation.h +++ b/src/include/nodes/relation.h @@ -490,8 +490,6 @@ typedef struct RelOptInfo /* default result targetlist for Paths scanning this relation */ struct PathTarget *reltarget; /* list of Vars/Exprs, cost, width */ - bool reltarget_has_non_vars; /* true if any expression in - * PathTarget is a non-Var */ /* materialization information */ List *pathlist; /* Path structures */ |