From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sun, 13 Mar 2005 19:59:40 +0000 (+0000)
Subject: Forgot that I had intended to replace division by masking in hash calculation.
X-Git-Tag: REL8_1_0BETA1~1224
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dffbbb3e5520ea87fc5484664d8b2f88c8dbcdf7;p=postgresql

Forgot that I had intended to replace division by masking in hash calculation.
---

diff --git a/src/backend/executor/nodeHash.c b/src/backend/executor/nodeHash.c
index c85755890a..ded952a43e 100644
--- a/src/backend/executor/nodeHash.c
+++ b/src/backend/executor/nodeHash.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/nodeHash.c,v 1.89 2005/03/06 22:15:04 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/nodeHash.c,v 1.90 2005/03/13 19:59:40 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -709,7 +709,8 @@ ExecHashGetBucketAndBatch(HashJoinTable hashtable,
 	if (nbatch > 1)
 	{
 		*bucketno = hashvalue % nbuckets;
-		*batchno = (hashvalue / nbuckets) % nbatch;
+		/* since nbatch is a power of 2, can do MOD by masking */
+		*batchno = (hashvalue / nbuckets) & (nbatch - 1);
 	}
 	else
 	{