Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New version tag "smart os only" #29

Open
fuson opened this issue Feb 17, 2013 · 1 comment
Open

New version tag "smart os only" #29

fuson opened this issue Feb 17, 2013 · 1 comment

Comments

@fuson
Copy link

fuson commented Feb 17, 2013

Hi, am i right that you are support your lib only for smart os? :) Sorry for this question, but it is critical for me cause i'm using your module in production.

@mcavage
Copy link
Owner

mcavage commented Feb 18, 2013

Hi,

No, it works everywhere, but there is a forked copy of the ZK C sources (courtesy of @yunong). We (Joyent) have been running ZK in production for some time now, and have found some really crippling problems in the C client related to "dropped sockets" - we had proposed a patch to the zookeeper trunk, which they didn't take, so we're just maintaing a fork, sadly. Since we use zkplus internally for everything that touches ZK and we only use smartos, we only did the work for building/maintaining our fork on smartos and tacked it into zkplus- on all other platforms zkplus will use the standard zookeeper 3.4.3 sources from node-zookeeper. The -smartos-only tag refers to the version that bundles in the forked zookeeper sources.

We'd be more than happy to share this if you were interested in porting that and/or revisiting with the ZK team

m

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants