search

RussellLuo / timingwheel

Golang implementation of Hierarchical Timing Wheels.
MIT License
660 stars 125 forks source link

Fix the deadlock issue between bucket.Add and bucket.Flush #28

Closed RussellLuo closed 4 years ago

RussellLuo commented 4 years ago

Fixed #25.

yiplee commented 4 years ago

这个 PR 可能引入了新的问题，之前运行没问题的项目更新了最新的 commit 之后出现了死锁的问题。

RussellLuo commented 4 years ago

这个 PR 可能引入了新的问题，之前运行没问题的项目更新了最新的 commit 之后出现了死锁的问题。

@yiplee 谢谢你的反馈！

我分析了下，修改后 Flush 函数中的 for 循环确实有问题，刚刚提交了一个新的修复 PR：#29 （等待你回复后 Merge :-)）

如果这个修复不能解决你的问题，欢迎新开一个 issue 讨论。