gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Eliminate eviction loops. And don't OOM so often with storage providers set

Browse Source 
Former-commit-id: 189674aea74c483bf0869050ecf7bf2b0b49f34e
This commit is contained in:
John Sully 2020-07-11 04:23:35 +00:00
parent 8256661b7a
commit a3cf9ce109
1 changed files with 18 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/evict.cpp
View File

@ -497,6 +497,7 @@ int freeMemoryIfNeeded(bool fPreSnapshot) {
int slaves = listLength(g_pserver->slaves);
const bool fEvictToStorage = !cserver.delete_on_evict && g_pserver->db[0]->FStorageProvider();
int result = C_ERR;
int ckeysFailed = 0;
/* When clients are paused the dataset should be static not just from the
* POV of clients not being able to write, but also from the POV of
@ -619,8 +620,16 @@ int freeMemoryIfNeeded(bool fPreSnapshot) {
{
// This key is in the storage so we only need to free the object
delta = (long long) zmalloc_used_memory();
db->removeCachedValue(bestkey);
delta -= (long long) zmalloc_used_memory();
if (db->removeCachedValue(bestkey)) {
delta -= (long long) zmalloc_used_memory();
ckeysFailed = 0;
}
else {
delta = 0;
ckeysFailed++;
if (ckeysFailed > 1024)
goto cant_free;
}
mem_freed += delta;
}
else
@ -679,6 +688,13 @@ int freeMemoryIfNeeded(bool fPreSnapshot) {
result = C_OK;
cant_free:
if (g_pserver->m_pstorageFactory)
{
if (mem_reported < g_pserver->maxmemory*1.2) {
return C_OK; // Allow us to temporarily go over without OOMing
}
}
if (!cserver.delete_on_evict && result != C_OK)
{
for (int idb = 0; idb < cserver.dbnum; ++idb)