> This comment certainly requires some changes.
Fixed.
> BTW, could you explain why init_table_size was two times less than
> max_table_size?
I have no clue. My best guess is that it was a reasonable thing to do in
the past. Then somebody changed a code and now there is little reason
to use init_table_size for partitioned tables.
> Why did you delete these two lines? I wonder if you should rewrite
> them instead?
```
MemSet(hctl, 0, sizeof(HASHHDR));
- hctl->nentries = 0;
- hctl->freeList = NULL;
```
These fields were initialized with zero values twice. It makes little
sense to me.
> As far as I understood, this number was obtained experimentally?
> Maybe you should note that in the comment.
These numbers are very platform specific and will be outdated very
soon. I recall that my code was criticized for including exact numbers
not a long time ago. So I suggest to keep this part as is.
> For example, if you have nelem=25 and partitions_number=6.
> 25 / 6 = 4. And then you allocate 24 nelems, while 1 is lost.
Agree. Fixed.
> Except mentioned notes, I suppose the patch is good enough
I guess I will mark this patch as "Ready for Committer" then.
