Re: [HACKERS] estimated_count() implementation

2006-10-22 Thread Volkan YAZICI
On Oct 21 05:09, Michael Fuhr wrote:
 I hadn't noticed the TODO item but about a year ago I posted a
 cursor_plan_rows() function and asked for comments.

Ah! I didn't see this.

 The only reply was from Tom, who said, Given how far off it
 frequently is, I can't believe that any of the people who ask for the
 feature would find this a satisfactory answer :-(

AFAIU, cursor_plan_rows() has some serious limitations like requiring to
be executed for a portal. I was planning to make estimated_count() work
for nodeAgg and custom calls too - as count() does.

But OTOH, Tom's complaints look like still applicable for my
estimated_count() too. Does this TODO need a little bit more
clarification or we can count is a redundant one?


Regards.

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


Re: [HACKERS] estimated_count() implementation

2006-10-22 Thread Simon Riggs
On Sun, 2006-10-22 at 12:07 +0300, Volkan YAZICI wrote:
 On Oct 21 05:09, Michael Fuhr wrote:
  I hadn't noticed the TODO item but about a year ago I posted a
  cursor_plan_rows() function and asked for comments.
 
 Ah! I didn't see this.
 
  The only reply was from Tom, who said, Given how far off it
  frequently is, I can't believe that any of the people who ask for the
  feature would find this a satisfactory answer :-(
 
 AFAIU, cursor_plan_rows() has some serious limitations like requiring to
 be executed for a portal. I was planning to make estimated_count() work
 for nodeAgg and custom calls too - as count() does.
 
 But OTOH, Tom's complaints look like still applicable for my
 estimated_count() too. Does this TODO need a little bit more
 clarification or we can count is a redundant one?

http://archives.postgresql.org/pgsql-hackers/2005-11/msg00943.php

Is the source of the TODO item, though please read the upthread messages
as to how we got there...

I think it would be a useful function...

-- 
  Simon Riggs 
  EnterpriseDB   http://www.enterprisedb.com



---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org


Re: [HACKERS] estimated_count() implementation

2006-10-22 Thread Tom Lane
Simon Riggs [EMAIL PROTECTED] writes:
 On Sun, 2006-10-22 at 12:07 +0300, Volkan YAZICI wrote:
 But OTOH, Tom's complaints look like still applicable for my
 estimated_count() too. Does this TODO need a little bit more
 clarification or we can count is a redundant one?

 http://archives.postgresql.org/pgsql-hackers/2005-11/msg00943.php

 Is the source of the TODO item, though please read the upthread messages
 as to how we got there...

I think there is a use-case for something like

select estimated_count('select * from ... where ...');

i.e. there are applications where a possibly-bad estimate is enough.
What I'm concerned about is the level of complaints from newbies who'll
expect it to be dead accurate all the time ...

BTW, you can build estimated_count() today in a few lines of plpgsql:

create or replace function estimated_count(text) returns float8 as $$
declare r text;
begin
  for r in execute 'explain ' || $1 loop
return substring(r from 'rows=([0-9]+) ');
  end loop;
end$$ language plpgsql strict;

I don't see that it really justifies any more work than that.

regards, tom lane

---(end of broadcast)---
TIP 6: explain analyze is your friend


Re: [HACKERS] estimated_count() implementation

2006-10-21 Thread Michael Fuhr
On Sat, Oct 21, 2006 at 11:44:19PM +0300, Volkan YAZICI wrote:
 I'm trying to implement estimated_count() function that's mentioned in
 the TODO list. First of all, I wanted to learn if this TODO item is
 still valid? I looked at the related -hackers discussions, does anybody
 want to say more sth related with the implementation?

I hadn't noticed the TODO item but about a year ago I posted a
cursor_plan_rows() function and asked for comments.  The only reply
was from Tom, who said, Given how far off it frequently is, I can't
believe that any of the people who ask for the feature would find
this a satisfactory answer :-(

http://archives.postgresql.org/pgsql-hackers/2005-11/msg00579.php
http://archives.postgresql.org/pgsql-hackers/2005-11/msg00580.php

-- 
Michael Fuhr

---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings