NotificationsYou must be signed in to change notification settings
Fork4.9k
Star17.7k

Commitd637a89

committed

Further twiddling of nodeHash.c hashtable sizing calculation.

On reflection, the submitted patch didn't really work to prevent therequest size from exceeding MaxAllocSize, because of the fact that we'dhappily round nbuckets up to the next power of 2 after we'd limited it tomax_pointers. The simplest way to enforce the limit correctly is toround max_pointers down to a power of 2 when it isn't one already.(Note that the constraint to INT_MAX / 2, if it were doing anything usefulat all, is properly applied after that.)

1 parent2647b24 commitd637a89Copy full SHA for d637a89

File tree

1 file changed

-1

lines changed

src/backend/executor
- nodeHash.c

1 file changed

-1

lines changed

`‎src/backend/executor/nodeHash.c`

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -396,6 +396,7 @@ ExecChooseHashTableSize(double ntuples, int tupwidth, bool useskew,`
`396`	`396`	`longhash_table_bytes;`
`397`	`397`	`longskew_table_bytes;`
`398`	`398`	`longmax_pointers;`
	`399`	`+longmppow2;`
`399`	`400`	`intnbatch;`
`400`	`401`	`intnbuckets;`
`401`	`402`	`inti;`
`@@ -463,7 +464,12 @@ ExecChooseHashTableSize(double ntuples, int tupwidth, bool useskew,`
`463`	`464`	`*/`
`464`	`465`	`max_pointers= (work_mem*1024L) /sizeof(HashJoinTuple);`
`465`	`466`	`max_pointers=Min(max_pointers,MaxAllocSize /sizeof(HashJoinTuple));`
`466`		`-/* also ensure we avoid integer overflow in nbatch and nbuckets */`
	`467`	`+/* If max_pointers isn't a power of 2, must round it down to one */`
	`468`	`+mppow2=1L <<my_log2(max_pointers);`
	`469`	`+if (max_pointers!=mppow2)`
	`470`	`+max_pointers=mppow2 /2;`
	`471`	`+`
	`472`	`+/* Also ensure we avoid integer overflow in nbatch and nbuckets */`
`467`	`473`	`/* (this step is redundant given the current value of MaxAllocSize) */`
`468`	`474`	`max_pointers=Min(max_pointers,INT_MAX /2);`
`469`	`475`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitd637a89

File tree

1 file changed

1 file changed

`‎src/backend/executor/nodeHash.c`

0 commit comments