Code review comment for lp:~rogpeppe/gozk/update-event-interface

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

Let's have a single thread on this topic, please.

As I answered to your email:

>> The point isn't C or Go.. the issue is simply that there's real
>> information coming through the pipe being trashed in the suggested
>> interface.
>
> there really isn't.

The protocol provides three details to the watch. Two of them are
being dropped on the floor in your branch and not reaching the event
listener. If you disagree with this we wont' be able to agree on
anything else around that change.

« Back to merge proposal