Re: Bug #758: show_planner_stats does not work for new PREPARE calls

From: Neil Conway <neilc(at)samurai(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: barry(at)xythos(dot)com, pgsql-bugs(at)postgresql(dot)org, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Bug #758: show_planner_stats does not work for new PREPARE calls
Date: 2002-09-17 00:45:33
Message-ID: 87vg55a1g2.fsf@mailbox.samurai.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-patches

Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> I'm not sure why NDirectFileRead/NDirectFileWrite are unused at the
> moment, but they used to be used; I think the correct response is to
> put back the missing counter increments, not rip out the counters.

Ok, fair enough. It's worth noting that they've been broken for a
while -- for example, the HashJoin counter increments were broken when
you comitted r1.20 of executor/nodeHashJoin.c in May of '99.

I've attached a revised patch that doesn't remove the counters (but
doesn't increment them either: I'm not sure of all the places where
the counter should be incremented).

Cheers,

Neil

--
Neil Conway <neilc(at)samurai(dot)com> || PGP Key ID: DB3C29FC

Attachment Content-Type Size
prepare_planner_stats-2.patch text/x-patch 2.0 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Ron Johnson 2002-09-17 00:56:42 Re: [BUGS] Postgres storing time in strange manner
Previous Message Neil Conway 2002-09-17 00:37:35 Re: Bug #770: 7.2.1 --> 7.2.2 =? Problems with speed

Browse pgsql-patches by date

  From Date Subject
Next Message Rod Taylor 2002-09-17 12:24:03 Re: Bug: COPY IN doesn't test domain constraints
Previous Message Tom Lane 2002-09-16 21:54:00 Bug: COPY IN doesn't test domain constraints