armink / FlashDB

An ultra-lightweight database that supports key-value and time series data | 一款支持 KV 数据和时序数据的超轻量级数据库
Apache License 2.0
1.92k stars 435 forks source link

疑似发现了fdb_kvdb.c的概念错误? #328

Open littlewhitezkz opened 3 weeks ago

littlewhitezkz commented 3 weeks ago

原文:fdb_kvdb.c

line1809        db->sector_cache_table[i].empty_kv = FAILED_ADDR;
line1810        db->sector_cache_table[i].addr = FDB_DATA_UNUSED;

也许应该修改为

line1809        db->sector_cache_table[i].empty_kv =FDB_DATA_UNUSED ;
line1810        db->sector_cache_table[i].addr = FAILED_ADDR;

并不影响使用。

armink commented 3 weeks ago

可以提个 PR 哈

littlewhitezkz commented 1 week ago

可以提个 PR 哈

朱工您写的没有问题,是我的理解失误。这两个概念分别是获取地址失败,和缓存表(Cache)该项未使用。