Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

[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 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 Herrerahttp://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

[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 allpaths.c?

> I thought OPTIMIZER_DEBUG showed us all the possible paths, not just the
> final plan.

Yeah, but we could repoint that code at pprint.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

[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
> 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?

I thought OPTIMIZER_DEBUG showed us all the possible paths, not just the
final plan.

-- 
  Bruce Momjian  http://momjian.us
  EnterpriseDB http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

[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.  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?

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] compile error with -DOPTIMIZER_DEBUG

[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@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

[HACKERS] compile error with -DOPTIMIZER_DEBUG

[Jan Urbański]

Hi,

ISTM that there's a superfluous curly brace in print_path (which only
gets compiled with -DOPTIMIZER_DEBUG.

Patch attached.

Jan
diff --git a/src/backend/optimizer/path/allpaths.c b/src/backend/optimizer/path/allpaths.c
index bfadcb0..6b2f86c 100644
--- a/src/backend/optimizer/path/allpaths.c
+++ b/src/backend/optimizer/path/allpaths.c
@@ -1449,7 +1449,6 @@ print_path(PlannerInfo *root, Path *path, int indent)
    ((mp->outersortkeys) ? 1 : 0),
    ((mp->innersortkeys) ? 1 : 0),
    ((mp->materialize_inner) ? 1 : 0));
-			}
 		}
 
 		print_path(root, jp->outerjoinpath, indent + 1);

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers