| From: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
|---|---|
| To: | Yeb Havinga <yebhavinga(at)gmail(dot)com> |
| Cc: | Alvaro Herrera <alvherre(at)commandprompt(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Fix for seg picksplit function |
| Date: | 2010-11-15 09:06:44 |
| Message-ID: | AANLkTikU-sO85aNTcKcjXdB6=WkVR7R5DEDoe+zY+nTp@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
With help of Oleg I found, that line "*left = *right = FirstOffsetNumber;"
was needed only for 7.X compatibility, and it isn't needed any more.
Also, I've replaced "i - 1" by "i - FirstOffsetNumber" in array filling.
I believe it's more correct way, because it'll work correctly in the case
when FirstOffsetNumber alters.
----
With best regards,
Alexander Korotkov.
| Attachment | Content-Type | Size |
|---|---|---|
| seg_picksplit_fix-0.3.patch | text/x-patch | 6.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Stehule | 2010-11-15 09:34:41 | Re: MULTISET and additional functions for ARRAY |
| Previous Message | Itagaki Takahiro | 2010-11-15 08:47:15 | Re: MULTISET and additional functions for ARRAY |