summaryrefslogtreecommitdiff
path: root/src/backend/access/hash
diff options
context:
space:
mode:
authorRobert Haas <rhaas@postgresql.org>2017-04-04 07:42:05 -0400
committerRobert Haas <rhaas@postgresql.org>2017-04-04 07:45:04 -0400
commitb38006ef6d1ba2f56cc4962ed17956b74c9fa0c4 (patch)
tree7359ac6f1bbb06f9b182acda61f26f493a784ee9 /src/backend/access/hash
parentea69a0dead5128c421140dc53fac165ba4af8520 (diff)
downloadpostgresql-b38006ef6d1ba2f56cc4962ed17956b74c9fa0c4.tar.gz
Fix formula in _hash_spareindex.
This was correct in earlier versions of the patch that lead to commit ea69a0dead5128c421140dc53fac165ba4af8520, but somehow got broken in the last version which I actually committed. Mithun Cy, per an off-list report from Ashutosh Sharma Discussion: http://postgr.es/m/CAD__OujbAwNU71v1y-RoQxZ8LZ6-V2UFTkex3v34MK6uZ3Xb5w@mail.gmail.com
Diffstat (limited to 'src/backend/access/hash')
-rw-r--r--src/backend/access/hash/hashutil.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/backend/access/hash/hashutil.c b/src/backend/access/hash/hashutil.c
index d679cf0aed..037582bd79 100644
--- a/src/backend/access/hash/hashutil.c
+++ b/src/backend/access/hash/hashutil.c
@@ -174,7 +174,8 @@ _hash_spareindex(uint32 num_bucket)
/* account for phases within current group */
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. */
return splitpoint_phases;