diff options
author | Viktor Söderqvist <viktor.soderqvist@est.tech> | 2023-01-20 17:45:29 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-01-20 18:45:29 +0200 |
commit | f3f6f7c0d66f136146a912e06c8fbe31ecfbc977 (patch) | |
tree | 246b79d2ca0ac0d5420ad6a86512702ae7957b75 /src/server.c | |
parent | b4123663c31aaf1e97f4dbd630cbd7b8d0e91e31 (diff) | |
download | redis-f3f6f7c0d66f136146a912e06c8fbe31ecfbc977.tar.gz |
Key as dict entry - memory optimization for sets (#11595)
If a dict has only keys, and no use of values, then a key can be stored directly in a
dict's hashtable. The key replaces the dictEntry. To distinguish between a key and
a dictEntry, we only use this optimization if the key is odd, i.e. if the key has the least
significant bit set. This is true for sds strings, since the sds header is always an odd
number of bytes.
Dict entries are used as a fallback when there is a hash collision. A special dict entry
without a value (only key and next) is used so we save one word in this case too.
This saves 24 bytes per set element for larges sets, and also gains some speed improvement
as a side effect (less allocations and cache misses).
A quick test adding 1M elements to a set using the command below resulted in memory
usage of 28.83M, compared to 46.29M on unstable.
That's 18 bytes per set element on average.
eval 'for i=1,1000000,1 do redis.call("sadd", "myset", "x"..i) end' 0
Other changes:
Allocations are ensured to have at least 8 bits alignment on all systems. This affects 32-bit
builds compiled without HAVE_MALLOC_SIZE (not jemalloc or glibc) in which Redis
stores the size of each allocation, after this change in 8 bytes instead of previously 4 bytes
per allocation. This is done so we can reliably use the 3 least significant bits in a pointer to
encode stuff.
Diffstat (limited to 'src/server.c')
-rw-r--r-- | src/server.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/server.c b/src/server.c index 910f299f3..8b4ec28e4 100644 --- a/src/server.c +++ b/src/server.c @@ -448,7 +448,8 @@ dictType setDictType = { NULL, /* val dup */ dictSdsKeyCompare, /* key compare */ dictSdsDestructor, /* key destructor */ - NULL /* val destructor */ + .no_value = 1, /* no values in this dict */ + .keys_are_odd = 1 /* an SDS string is always an odd pointer */ }; /* Sorted sets hash (note: a skiplist is used in addition to the hash table) */ @@ -471,9 +472,9 @@ dictType dbDictType = { dictSdsDestructor, /* key destructor */ dictObjectDestructor, /* val destructor */ dictExpandAllowed, /* allow to expand */ - dbDictEntryMetadataSize, /* size of entry metadata in bytes */ - dbDictMetadataSize, /* size of dict metadata in bytes */ - dbDictAfterReplaceEntry /* notify entry moved/reallocated */ + .dictEntryMetadataBytes = dbDictEntryMetadataSize, + .dictMetadataBytes = dbDictMetadataSize, + .afterReplaceEntry = dbDictAfterReplaceEntry }; /* Db->expires */ |