Re: Double sorting split patch - Mailing list pgsql-hackers

From Heikki Linnakangas
Subject Re: Double sorting split patch
Date
Msg-id 4E7B1A7C.80103@enterprisedb.com
Whole thread Raw
In response to Re: Double sorting split patch  (Alexander Korotkov <aekorotkov@gmail.com>)
Responses Re: Double sorting split patch
List pgsql-hackers
> !         /*
> !          * Calculate delta between penalties of join "common entries" to
> !          * different groups.
> !          */
> !         for (i = 0; i < commonEntriesCount; i++)
>           {
> !             double        lower,
> !                         upper;
> !
> !             box = DatumGetBoxP(entryvec->vector[commonEntries[i].index].key);
> !             if (context.dim == 0)
> !             {
> !                 lower = box->low.x;
> !                 upper = box->high.x;
> !             }
> !             else
> !             {
> !                 lower = box->low.y;
> !                 upper = box->high.y;
> !             }
> !             commonEntries[i].delta = Abs(box_penalty(leftBox, box) -
> !                                          box_penalty(rightBox, box));
>           }

'lower' and 'upper' are not used for anything in the above. Is that just 
dead code that can be removed, or is there something missing that should 
be using them?

--   Heikki Linnakangas  EnterpriseDB   http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: Alexander Korotkov
Date:
Subject: Re: WIP: SP-GiST, Space-Partitioned GiST
Next
From: Kyotaro HORIGUCHI
Date:
Subject: Re: [v9.2] make_greater_string() does not return a string in some cases