From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, Jan Urbański <wulczer(at)wulczer(dot)org>, Postgres - Hackers <pgsql-hackers(at)postgresql(dot)org>, Tom Raney <raneyt(at)cecs(dot)pdx(dot)edu> |
Subject: | Re: compile error with -DOPTIMIZER_DEBUG |
Date: | 2009-12-02 15:02:17 |
Message-ID: | 20091202150217.GA6276@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane wrote:
> Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> 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. Personally I always use pprint()
> when I'm interested in a printout of a plan tree. Is anyone actually
> using the printout code in allpaths.c?
Maybe Tom Raney's Visual Planner thing?
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support
From | Date | Subject | |
---|---|---|---|
Next Message | Tim Bunce | 2009-12-02 15:04:01 | Initial refactoring of plperl.c - rebased [PATCH] |
Previous Message | Andrew Dunstan | 2009-12-02 14:48:33 | Re: YAML Was: CommitFest status/management |