[sheepdog] [PATCH v4 5/5] zookeeper: handle session timeout for all zookeeper operations

Kai Zhang kyle at zelin.io
Tue Jun 18 09:59:28 CEST 2013


On Jun 18, 2013, at 3:43 PM, Liu Yuan <namei.unix at gmail.com> wrote:

> I am wondering why this is necessary. All the handlers are called in the
> main thread include handling of reconnect. So basically, whenever you
> reach here, zk_flying_ops will be 0, no?
> 
> Based on my understanding, this patch is completely unnecessary.

You are right. the 'zk_flying_ops' is not necessary. I will remove it.

However, I think this patch also removes unnecessary panic when zk_* APIs returns
ZINVALIDSTATE.
So we should include it.

Thanks,
Kyle
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wpkg.org/pipermail/sheepdog/attachments/20130618/ef6a163c/attachment-0004.html>


More information about the sheepdog mailing list