lp://staging/~niemeyer/gozk/watches-die-on-reconnection

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/gozk/zookeeper at revision 35

The Go Language Gophers: Pending requested 2012-06-04

Diff: 89 lines (+26/-19)

2 files modified

zk.go (+6/-5)
zk_test.go (+20/-14)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

35. By Gustavo Niemeyer on 2012-06-04

Any session event on a watcher now fires and closes the watch.

Also test that closing the connection closes the watch.

34. By Gustavo Niemeyer on 2012-04-05

Fixed example which was completely broken.

33. By Roger Peppe on 2012-03-15

zookeeper: make error messages more informative.

R=niemeyer
CC=
https://codereview.appspot.com/5835045

32. By Roger Peppe on 2012-03-14

zookeeper: allow CreateServer to work in an existing directory.

It only succeeds if the directory is empty, for instance
when it's created by ioutil.TempDir.

Also add Server.Addr method and change the NotRunning
error to ErrNotRunning.

R=fwereade, niemeyer
CC=
https://codereview.appspot.com/5787068

31. By Roger Peppe on 2012-03-08

zookeeper: make Conn.Close safe to call concurrently with other operations.

I am concerned at how complex the test for this is,
relative to the simplicity of the actual code change.
(and it's also potentially fragile on a heavily loaded machine).
That said, I can't think of another way to test the
operations directly, so I'm leaving it in.
Better suggestions welcome.

R=niemeyer, fwereade
CC=
https://codereview.appspot.com/5699093

30. By Gustavo Niemeyer on 2012-02-23

Fix after latest changes in Go's tip.

R=rog
CC=
https://codereview.appspot.com/5684066

29. By Gustavo Niemeyer on 2012-02-15

Fix Exists.

R=fwereade
CC=
https://codereview.appspot.com/5673057

28. By Gustavo Niemeyer on 2012-02-15

Use int32 rather than int for stat fields.

R=fwereade, TheMue, rog
CC=
https://codereview.appspot.com/5675059

27. By Gustavo Niemeyer on 2012-02-15

Fix support for weekly Go release.

R=TheMue
CC=
https://codereview.appspot.com/5656063

26. By Roger Peppe on 2012-02-13

Add some comments to Event and Stat types.

Also change Stat.CTime and Stat.MTime to return time.Time.

R=niemeyer
CC=
https://codereview.appspot.com/5650075