keithchew
keithchew
This is just a preliminary bug report, as I am still trying to track down the root cause. Steps are simple: (1) Start with fresh KeyDB instance (2) Load 6mil...
**Describe the bug** There are 2 places where KeyDB does a KeySpaceEvent: in db.cpp (for flash): ``` moduleNotifyKeyspaceEvent(NOTIFY_LOADED, "loaded", &keyobj, *(int *)data); ``` and in rdb.cpp (for RDB): ``` moduleNotifyKeyspaceEvent(NOTIFY_LOADED,...
**Describe the bug** With a new DB: ``` set test 222 EX 5 ``` Logging the keys from expire.cpp, ie: ``` keys = db->getStorageCache()->getExpirationCandidates(ACTIVE_EXPIRE_CYCLE_LOOKUPS_PER_LOOP); for (std::string key : keys) {...
**Describe the bug** With flash enabled on v6.3.4, I wrote a script that loads up the DB with > 3mil keys and majority having TTL expiring in 24hrs. Comes the...
In rocksdbfactory.cpp, we have: ``` options.compression = rocksdb::kNoCompression; ``` but it should be: ``` options.compression = rocksdb::CompressionType::kNoCompression; ``` Enabling logging in rocksdb will confirm that the former will not disable...
**Crash report** I wrote a test module to perform RedisModule_Scan, and if the data is large enough or if this is called enough times, I get a crash. Module snippet...
I upgraded from v4.0.8 to v5.0.0 on a mainnet node, and on startup I am getting this error in logs: ``` [2024-02-22 20:49:29] INFO db: Cleaning up dirty states count=1...
Testing in async_falsh branch, in db.cpp, one change is to reset m_numexpires in removeAllCachedValues(): ``` void redisDbPersistentData::removeAllCachedValues() ... m_numexpires = 0; } ``` When the server fails to evict any...
Testing on v6.3.4 (with flash), but this bug should be present in previous versions as well. Steps: - master and slave both in sync - ensure disk space on master...
Testing on v6.3.4, when a slave starts up, it performs a full sync with the master. After receiving the RDB file from the master, it attempts to load the dataset...