diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-02-25 03:30:38 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-02-25 03:30:38 +0000 |
| commit | e549722a8b2fcc5b2ad9a9bd53948466ea5dae26 (patch) | |
| tree | a55ae57bb515b353f70a8321a0751ed5aca40044 /src/backend/nodes/outfuncs.c | |
| parent | 7380b6384b5a0b25fa7dfa7447dbf95ff1c4b8da (diff) | |
| download | postgresql-e549722a8b2fcc5b2ad9a9bd53948466ea5dae26.tar.gz | |
Get rid of the rather fuzzily defined FlattenedSubLink node type in favor of
making pull_up_sublinks() construct a full-blown JoinExpr tree representation
of IN/EXISTS SubLinks that it is able to convert to semi or anti joins.
This makes pull_up_sublinks() a shade more complex, but the gain in semantic
clarity is worth it. I still have more to do in this area to address the
previously-discussed problems, but this commit in itself fixes at least one
bug in HEAD, as shown by added regression test case.
Diffstat (limited to 'src/backend/nodes/outfuncs.c')
| -rw-r--r-- | src/backend/nodes/outfuncs.c | 16 |
1 files changed, 1 insertions, 15 deletions
diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c index 5dc9db98bf..c68dea8a2c 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.352 2009/02/06 23:43:23 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.353 2009/02/25 03:30:37 tgl Exp $ * * NOTES * Every node type that can appear in stored rules' parsetrees *must* @@ -1631,17 +1631,6 @@ _outInnerIndexscanInfo(StringInfo str, InnerIndexscanInfo *node) } static void -_outFlattenedSubLink(StringInfo str, FlattenedSubLink *node) -{ - WRITE_NODE_TYPE("FLATTENEDSUBLINK"); - - WRITE_ENUM_FIELD(jointype, JoinType); - WRITE_BITMAPSET_FIELD(lefthand); - WRITE_BITMAPSET_FIELD(righthand); - WRITE_NODE_FIELD(quals); -} - -static void _outPlaceHolderVar(StringInfo str, PlaceHolderVar *node) { WRITE_NODE_TYPE("PLACEHOLDERVAR"); @@ -2660,9 +2649,6 @@ _outNode(StringInfo str, void *obj) case T_InnerIndexscanInfo: _outInnerIndexscanInfo(str, obj); break; - case T_FlattenedSubLink: - _outFlattenedSubLink(str, obj); - break; case T_PlaceHolderVar: _outPlaceHolderVar(str, obj); break; |