node-zkplus icon indicating copy to clipboard operation
node-zkplus copied to clipboard

New version tag "smart os only"

Open fuson opened this issue 12 years ago • 1 comments

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.

fuson avatar Feb 17 '13 15:02 fuson

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

mcavage avatar Feb 18 '13 16:02 mcavage