Heikki Linnakangas <[EMAIL PROTECTED]> writes:
> Tom Lane wrote:
>> This will not work, unless we change the planner --- the original quals
>> aren't necessarily there in some corner cases (partial indexes, if
>> memory serves).

> This is only for bitmap scans, which *do* always have the original quals 
>    available in the executor (BitmapHeapScanState.bitmapqualorig). 
> That's because we have to recheck the original conditions when the 
> bitmap goes lossy.

Yeah, but the index AM has to support regular indexscans too, and those
are not prepared for runtime lossiness determination; nor am I
particularly willing to add that.

> With the unapplied GIT patch, the index doesn't store the index key of 
> every tuple.

I thought the design was to eliminate *duplicate* keys from the index.
Not to lose data.

                        regards, tom lane

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

Reply via email to