Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

2009-12-02 Thread Alvaro Herrera
Tom Lane wrote: > Heikki Linnakangas writes: > > Jan Urbański wrote: > >> ISTM that there's a superfluous curly brace in print_path (which only > >> gets compiled with -DOPTIMIZER_DEBUG. > > > Thanks, committed. > > You know, the last couple of times I've touched that code, I've been > wondering

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

2009-11-29 Thread Tom Lane
Bruce Momjian writes: > Tom Lane wrote: >> You know, the last couple of times I've touched that code, I've been >> wondering why we bother to maintain it. Personally I always use pprint() >> when I'm interested in a printout of a plan tree. Is anyone actually >> using the printout code in allpat

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

2009-11-29 Thread Bruce Momjian
Tom Lane wrote: > Heikki Linnakangas writes: > > Jan Urba??ski wrote: > >> ISTM that there's a superfluous curly brace in print_path (which only > >> gets compiled with -DOPTIMIZER_DEBUG. > > > Thanks, committed. > > You know, the last couple of times I've touched that code, I've been > wonderin

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

2009-11-22 Thread Tom Lane
Heikki Linnakangas writes: > Jan Urbański wrote: >> ISTM that there's a superfluous curly brace in print_path (which only >> gets compiled with -DOPTIMIZER_DEBUG. > Thanks, committed. You know, the last couple of times I've touched that code, I've been wondering why we bother to maintain it. P

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

2009-11-22 Thread Heikki Linnakangas
Jan Urbański wrote: > ISTM that there's a superfluous curly brace in print_path (which only > gets compiled with -DOPTIMIZER_DEBUG. > > Patch attached. Thanks, committed. -- Heikki Linnakangas EnterpriseDB http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers