> On Fri, Sep 4, 2015 at 10:03 AM, Tatsuo Ishii <is...@postgresql.org> wrote:
>>
>> I have looked into your patch and am a little bit worried because it
>> calls RelationGetNumberOfBlocks() which is an expensive function.
>> I think there are two ways to avoid it.
>>
>> 1) fall back to your former patch. However it may break existing
>>    behavior what you said. I think there's very little chance it
>>    actually happens because IndexBuildHeapRangeScan() is only used by
>>    brin (I confirmed this). But Alvaro said some patches may be it's
>>    customers. Robert, Amit, Can you please confirm this?
>>
> 
> In earlier version of parallel_seqscan patch, heap_setscanlimits() was being
> used, but now altogether a different mechanism is used.

Thank you for the confirmation!

Best regards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp


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

Reply via email to