Re: [HACKERS] enable_material patch

2010-04-18 Thread Robert Haas
On Sun, Apr 18, 2010 at 4:16 PM, Tom Lane wrote: > Robert Haas writes: >> Here's a patch to add enable_material, per previous discussion.  I >> still think we should add enable_joinremoval also, but there wasn't a >> clear consensus for that. > >> I'd appreciate it if someone could check this ove

Re: [HACKERS] enable_material patch

2010-04-18 Thread Tom Lane
Robert Haas writes: > Here's a patch to add enable_material, per previous discussion. I > still think we should add enable_joinremoval also, but there wasn't a > clear consensus for that. > I'd appreciate it if someone could check this over for sanity - like, > did I get all the places where mat

[HACKERS] enable_material patch

2010-04-17 Thread Robert Haas
Here's a patch to add enable_material, per previous discussion. I still think we should add enable_joinremoval also, but there wasn't a clear consensus for that. I'd appreciate it if someone could check this over for sanity - like, did I get all the places where materialize nodes can be created?