rewriteaof 存在死锁的可能 about godis HOT 7 CLOSED

明白， 还真有点难处理。我把 rewrite buffer 改到临时文件好了

from godis.

提供一种解决方法，如下：

from godis.

这个方案没有解决 aofRewriteBuffer <- cmd 会阻塞的问题。handleAof 被阻塞后，db.aofChan 很快也会填满，进而阻塞主协程的 addAof 函数， 最终导致所有写命令被阻塞，同时 goroutine 数量迅速上升。我觉得还是从源头上解决 aofRewriteBuffer 会被填满的问题比较好

from godis.

是的，上面的方案只是解决了死锁的问题。要解决aofRewriteBuffer阻塞的问题，还有个方法：

1. 直接去掉aofRewriteBuffer，handleAof方法中，收到aofChan的数据后，获取锁，不再同步一份数据到aofRewriteBuffer。
2. startRewrite方法中，已经获取到了aof 文件的filesize A，这个filesize在finishRewrite方法中可以用上。
3. finishRewrite方法中，获取到锁之后，handleAof方法无法获取到锁，也就没办法继续往aof文件中写。finishRewrite方法此时再对aof文件sync落盘，此时从filesize A 到aof 文件尾，就是在重写aof文件过程中，处理的请求，直接将这些操作读出来写到临时文件，最后mv就行了

from godis.

好主意

from godis.

读取aof文件内容，加载内容到临时DB对象

如果当前数据库中有1G的数据，再加载一份AOF文件岂不是内存又多占用了1G？

from godis.

fixed in 48ba261

from godis.