Home

Welcome to the go-zookeeper wiki!

Resolved Upstream Pull Requests

The underlying issue or intent of some pull requests has been fixed or obviated by other changes. Those are listed here so we don't try to unnecessarily re-fix issues.

• https://github.com/samuel/go-zookeeper/pull/68
• https://github.com/samuel/go-zookeeper/pull/129
• https://github.com/samuel/go-zookeeper/pull/140
• https://github.com/samuel/go-zookeeper/pull/141
• https://github.com/samuel/go-zookeeper/pull/142
• https://github.com/samuel/go-zookeeper/pull/183

Integrated Upstream Pull Requests

These changes have been applied more or less as-is.

• https://github.com/samuel/go-zookeeper/pull/146
• https://github.com/samuel/go-zookeeper/pull/188
• https://github.com/samuel/go-zookeeper/pull/206

Closed Upstream Issues

• https://github.com/samuel/go-zookeeper/issues/157
  • Uncache DNS by using WithHostProvider(&SimpleDNSHostProvider{})

Pending Upstream Issues

Interesting Pull Requests

Remove Watch API

These are good ideas, but fundamentally incompatible with this version of the API. Defer integration until we have semantic versioning.

• https://github.com/samuel/go-zookeeper/pull/110
• https://github.com/samuel/go-zookeeper/pull/136

Locking Example

• https://github.com/samuel/go-zookeeper/pull/168

Race Fix

• https://github.com/samuel/go-zookeeper/pull/186
  • Unclear this is still relevant, go test -race seems to run OK.

DNS Fix

• https://github.com/samuel/go-zookeeper/pull/191
  • Relevant to DNS re-resolve, but seems complex. Probably better to focus on #206.