sorintlab / stolon

PostgreSQL cloud native High Availability and more.
https://talk.stolon.io
Apache License 2.0
4.63k stars 444 forks source link

Use Stolon with Zookeeper #542

Open eolix opened 6 years ago

eolix commented 6 years ago

Submission type

Stolon version

Latest

Enhancement Description

The framework itself already supports Zookeeper, but there is no documentation regarding how to use it. This is particularly useful to be able to use it on DC/OS (or Mesos+Marathon), as it comes with ZK built in.

sgotti commented 6 years ago

@eolix Currently stolon doesn't support zookeeper. If with "framework" you mean libkv, the main issue to implement a zookeeper store is the "unmaintained" state of libkv.

Another (probably better) solution will be to implement a store without relying on libkv (like done for the etcdv3 store).

Additionally the test suite must be enhanced to also test with zookeper.

I'll happily review PRs if someone is willing to develop and maintain it. Another option will be to sponsor its development/maintenance: https://sorintoss.io)

siepkes commented 5 years ago

Zookeeper seems to be fully supported now with libkv (see README):

It is especially useful if you plan to support consul, etcd and zookeeper using the same codebase.

I might take a stab at this at some point since I would like to standardize on a single K/V store in our infra.

jackin853 commented 1 year ago

When will this be supported? Do you have a plan? @siepkes @sgotti

siepkes commented 1 year ago

@jackin853 I've never gotten around to it because for us it was a nice to have. The issue still has the "help wanted" label assigned to it so you might want to take a shot at it yourself and submit a PR.

jackin853 commented 1 year ago

@jackin853 I've never gotten around to it because for us it was a nice to have. The issue still has the "help wanted" label assigned to it so you might want to take a shot at it yourself and submit a PR.

Sorry, mainly I haven’t studied the source code, and I don’t know how to modify it yet. @siepkes @eolix