upyun / lua-resty-sync

Synchronizing data based on version changes
41 stars 6 forks source link

when equal version, instead of fetching from shm #2

Closed fankeke closed 6 years ago

fankeke commented 6 years ago

version should be play it's role

plus: self.owner may not be accurate when there are mutil timer task in on syner instance , so delete it .

tokers commented 6 years ago

@fankeke 这个 PR 有解决我们之前讨论的 https://github.com/upyun/lua-resty-sync/issues/1 这里的 timer 唯一性的问题吗?

resty-sync 应该要能保证 timer 的唯一性,我想把 new 函数的使用阶段改为 init 就行了。

tokers commented 6 years ago

@fankeke 这个 PR 有解决我们之前讨论的 https://github.com/upyun/lua-resty-sync/issues/1 这里的 timer 唯一性的问题吗?

resty-sync 应该要能保证 timer 的唯一性,我想把 new 函数的使用阶段改为 init 就行了。

fankeke commented 6 years ago

@tokers 暂时没有,这个PR是 解决: 每次获取data,不用从shm中去获取,应该更新其version。

另外crash worker后timer的唯一性问题,如果放在init阶段,感觉使用上有些不方便,最好能够在init_worker阶段搞定。

应该有其他办法。

tokers commented 6 years ago

唯一性的问题,放在 init_worker 处理的话,给每个实例增加一个变量标示即可。

tokers commented 6 years ago

这个 PR 的 idea 不错,我晚点会看下。

tokers commented 6 years ago

@fankeke 代码我看了下没问题,还请麻烦加一下测试 :)

fankeke commented 6 years ago

哦,忘了,待会加上来。

fankeke commented 6 years ago

@tokers 加了test case

fankeke commented 6 years ago

@tokers please have a look again

tokers commented 6 years ago

@fankeke Merged, Thanks.