On Tue, Apr 4, 2017 at 9:18 AM, Robert Haas <robertmh...@gmail.com> wrote:
> Committed.

Thanks Robert,

And also sorry, one unfortunate thing happened in the last patch while
fixing one of the review comments a variable disappeared from the

        splitpoint_phases +=
-               (((num_bucket - 1) >> (HASH_SPLITPOINT_PHASE_BITS + 1)) &
+               (((num_bucket - 1) >>
+                 (splitpoint_group - (HASH_SPLITPOINT_PHASE_BITS + 1))) &
                 HASH_SPLITPOINT_PHASE_MASK);   /* to 0-based value. */

I wanted most significant 3 bits. And while fixing comments in patch11
I unknowingly somehow removed splitpoint_group from the equation.
Extremely sorry for the mistake. Thanks to Ashutosh Sharma for
pointing the mistake.

Thanks and Regards
Mithun C Y
EnterpriseDB: http://www.enterprisedb.com

Attachment: _hash_spareindex_defect.patch
Description: Binary data

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

Reply via email to