Re: [HACKERS] Improve code in tidbitmap.c

2013-11-15 Thread Tom Lane
"Etsuro Fujita" writes: > I'd like to do the complementary explanation of this. > In tbm_union_page() and tbm_intersect_page() in tidbitmap.c, WORDS_PER_PAGE > is used in the scan of a lossy chunk, instead of WORDS_PER_CHUNK, where > these macros are defined as: > /* number of active words for an

Re: [HACKERS] Improve code in tidbitmap.c

2013-11-14 Thread David Rowley
On Fri, Nov 15, 2013 at 12:50 AM, Etsuro Fujita wrote: > > I think that, as a secondary effect of the patch, the scan would be > performed a bit efficiently. > > Hi Etsuro, Would you be able to supply some benchmarks of a work load which benefits from this change? Something which compares patche

Re: [HACKERS] Improve code in tidbitmap.c

2013-11-14 Thread Etsuro Fujita
I'd like to do the complementary explanation of this. In tbm_union_page() and tbm_intersect_page() in tidbitmap.c, WORDS_PER_PAGE is used in the scan of a lossy chunk, instead of WORDS_PER_CHUNK, where these macros are defined as: /* number of active words for an exact page: */ #define WORDS_PER_