citusdata / citusdata/postgresql-hll

Memory usage 100X what it should be for HashAgg

Open
#92 0 comments 4 reactions 0 assignees View on GitHub
Dominant language
C
Stars
1.2k
Forks
115
PR merge metrics
No merged PRs in 30d

Description

The transition state values are occupying 128K when they should be about 1.2K (for default values of ``regsize`` and ``log2m``).

This happens because of: https://github.com/citusdata/postgresql-hll/blob/master/src/hll.c#L3115 allocating ``sizeof(multiset_t)``, which is sized to be worse than the worst case. This allocated value is then returned by ``hll_add_trans0``.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.