Re: GIN improvements part 1: additional information - Mailing list pgsql-hackers

From Antonin Houska
Subject Re: GIN improvements part 1: additional information
Date
Msg-id 51CEC07C.4000408@gmail.com
Whole thread Raw
In response to Re: GIN improvements part 1: additional information  (Heikki Linnakangas <hlinnakangas@vmware.com>)
List pgsql-hackers
On 06/29/2013 11:00 AM, Heikki Linnakangas wrote:
> On 27.06.2013 17:20, Antonin Houska wrote:
>> I was curious about the new layout of the data page, so I spent a while
>> looking into the code.
>> It's interesting, but I suspect 2 things are not o.k.:
>>
>> * gindatapage.c:dataIsEnoughSpace() - 'i++' in the for loop should
>> probably be 'j++', otherwise it loops forever
>
> Hmm. It won't loop forever, i is counting the number of entries that 
> fit on the page, while j is used to loop through the items being 
> added. However, i isn't actually used for anything (and isn't 
> initialized either), so it's just dead code.
>
> - Heikki
You're right. While thinking about possible meaning of the 'i' I didn't 
notice that j++ is in the 'for' construct. Stupid mistake on my side.

Tony



pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: proposal: simple date constructor from numeric values
Next
From: "Erik Rijkers"
Date:
Subject: Re: psql and pset without any arguments