Teodor Sigaev <[EMAIL PROTECTED]> writes:
>> We could probably fix that by adding it to the stats structs that are
>> passed around during VACUUM.  I'd rather not hardwire a GIN special case
>> in vacuum.c as per your "quick hack".

> ok.   amskipcheck?

Oh, I was thinking of having VACUUM put the heap tuple count into the
struct and then amvacuumcleanup could copy it over to the index tuple
count.  A "skipcheck" flag only solves the cosmetic problem of not
getting the warning, it doesn't fix things so that the correct count
ends up in the index's reltuples entry.

>> Actually, does clustering on *any* current index type except btree make
>> sense?  None of them have semantically interesting index ordering

> I don't know about hash index, but for GiST clustering can speed up query's 
> execution.

OK, in that case we'd better add a real amclusterable flag to pg_am,
rather than assuming amorderstrategy can be used to decide.

> So, two columns about clustering?
> amclustered
> amclusterable

Huh?  Why two?  Either you are allowed to cluster on indexes of this
type, or you're not.  I don't see the point of any other distinction.

                        regards, tom lane

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

Reply via email to