Compare commits

..

No commits in common. "master" and "v1.0.4" have entirely different histories.

7 changed files with 48 additions and 93 deletions

View File

@ -30,21 +30,24 @@ function setup(opts) {
var crypto = require('crypto');
var methods = getMethods(opts.instance, opts.methods);
var token = crypto.randomBytes(16).toString('hex');
var msgPrefix = 'cluster-rpc.' + opts.name;
var rpcPrefix = msgPrefix + '.rpc';
var resultPrefix = msgPrefix + '.result';
var initPrefix = msgPrefix + '.init';
var inst = opts.instance;
var prefixes = require('./prefixes.js').create(opts); // uses opts.name
opts.master = opts.master || require('./process/master').create(prefixes);
opts.master = opts.master || require('./process/master').create();
opts.master.on('connection', function (w) {
if (opts.debug) { console.log('[cluster-rpc] [master] worker connected'); }
//console.log('debug w: worker connection');
w.send({
methods: methods
, _token: token
, type: prefixes.init
, type: initPrefix
});
w.on('message', function (cmd) {
if (0 !== (cmd.type||'').indexOf(prefixes.root)) {
if (0 !== (cmd.type||'').indexOf(msgPrefix)) {
//console.log('debug w: got unknown message type');
return;
}
@ -59,7 +62,7 @@ function setup(opts) {
}
switch (cmd.type) {
case prefixes.rpc:
case rpcPrefix:
cmd.args.push(function callback() {
// args is probably err, data in most cases
var args = Array.prototype.slice.call(arguments);
@ -69,7 +72,7 @@ function setup(opts) {
, id: cmd.id
//, this: this
, _token: token
, type: prefixes.result
, type: resultPrefix
});
});
@ -86,14 +89,12 @@ function setup(opts) {
}
module.exports.create = function (opts) {
if (opts.debug) { console.log('[cluster-rpc] [master] create'); }
var cluster = require('cluster');
var PromiseA = opts.PromiseA || global.Promise || require('bluebird');
var init = false;
opts._promise = PromiseA.resolve(opts.instance);
opts._promise.addWorker = function (w) {
if (opts.debug) { console.log('[cluster-rpc] [master] addWorker wrapper'); }
if (!init) {
init = true;
setup(opts);
@ -102,7 +103,6 @@ module.exports.create = function (opts) {
};
if (false !== opts.addOnFork) {
if (opts.debug) { console.log('[cluster-rpc] [master] -- will call addWorker on each fork'); }
cluster.on('fork', opts._promise.addWorker);
}

View File

@ -1,15 +1,14 @@
{
"name": "cluster-rpc",
"version": "1.0.7",
"version": "1.0.4",
"description": "A simple way to wrap a single-instance module to enable it to work with node cluster.",
"homepage": "https://git.coolaj86.com/coolaj86/cluster-rpc.js",
"main": "index.js",
"scripts": {
"test": "node test.js"
},
"repository": {
"type": "git",
"url": "git+https://git.coolaj86.com/coolaj86/cluster-rpc.js.git"
"url": "git+ssh://git@github.com/coolaj86/cluster-rpc.git"
},
"keywords": [
"cluster",
@ -18,6 +17,7 @@
"author": "AJ ONeal <coolaj86@gmail.com> (https://coolaj86.com/)",
"license": "(MIT OR Apache-2.0)",
"bugs": {
"url": "https://git.coolaj86.com/coolaj86/cluster-rpc.js/issues"
}
"url": "https://github.com/coolaj86/cluster-rpc/issues"
},
"homepage": "https://github.com/coolaj86/cluster-rpc#readme"
}

View File

@ -1,18 +0,0 @@
'use strict';
module.exports.create = function (opts) {
//var msgPrefix = 'cluster-rpc.' + opts.name;
//var rpcPrefix = msgPrefix + '.rpc';
//var resultPrefix = msgPrefix + '.result';
//var initPrefix = msgPrefix + '.init';
var root = 'com.daplie.cluster-rpc.' + (opts.name ? opts.name + '.' : '');
return {
root: root
, rpc: root + 'rpc'
, result: root + 'result'
, init: root + 'init'
, connect: root + 'connect'
// TODO the things that are using this should probably accept opts
, debug: opts.debug
};
};

View File

@ -1,45 +1,27 @@
'use strict';
module.exports.create = function (prefixes) {
if (prefixes.debug) { console.log('[cluster-rpc] master created'); }
module.exports.create = function () {
var m = new (require('events').EventEmitter)();
m.addWorker = function (worker) {
if (prefixes.debug) { console.log('[cluster-rpc] [master] adding worker'); }
m._workers = [];
var w = new (require('events').EventEmitter)();
function emitConnection() {
if (w.__online) {
return;
}
w.__online = true;
m.emit('connection', w);
}
worker.on('online', function () {
if (prefixes.debug) { console.log('[cluster-rpc] [master] worker came online, at fork'); }
emitConnection();
//console.log('debug mw: worker is up')
m.emit('connection', w);
});
worker.on('message', function (data) {
if (prefixes.connect === data.type) {
if (prefixes.debug) { console.log('[cluster-rpc] [master] worker connected, manually'); }
emitConnection();
return;
}
if (prefixes.debug) { console.log('[cluster-rpc] [master] worker sent message', data); }
//console.log('debug mw: worker sends message', data)
w.emit('message', data);
});
w.send = function (data) {
if (prefixes.debug) { console.log('[cluster-rpc] [master] sending', data); }
worker.send(data);
};
// TODO remove workers that exit
m._workers.push(w);
};

View File

@ -1,7 +1,6 @@
'use strict';
module.exports.create = function (process, prefixes) {
if (prefixes.debug) { console.log('[cluster-rpc] worker created'); }
module.exports.create = function (process) {
var w = new (require('events').EventEmitter)();
process.on('message', function (data) {
@ -12,10 +11,5 @@ module.exports.create = function (process, prefixes) {
process.send(data);
};
// if this were a web / unix socket there would be a 'connection' event
// emulating this is useful since the worker may create its cluster rpc
// at any time, (which means it may miss the 'fork' event)
w.send({ type: prefixes.connect });
return w;
};

37
test.js
View File

@ -2,18 +2,19 @@
var cluster = require('cluster');
var crpc;
var db = {
get: function (key, cb) {
cb(null, db[key]);
}
, put: function (key, val, cb) {
db[key] = val;
cb(null);
}
};
function runMaster() {
var db = {
get: function (key, cb) {
cb(null, db[key]);
}
, put: function (key, val, cb) {
db[key] = val;
if (cb) { cb(null); }
}
};
if (cluster.isMaster) {
crpc = require('./master').create({
instance: db
@ -25,9 +26,10 @@ function runMaster() {
});
cluster.fork();
}
function runWorker() {
}
else {
crpc = require('./worker').create({
name: 'foo-level'
@ -36,17 +38,6 @@ function runWorker() {
}
if (cluster.isMaster) {
runMaster();
}
else {
runWorker();
}
crpc.then(function (db) {
setTimeout(function () {

View File

@ -9,8 +9,11 @@ module.exports.create = function (opts) {
var crypto = require('crypto');
var token = null;
var inst = {};
var prefixes = require('./prefixes.js').create(opts);
var ws = opts.worker = opts.worker || require('./process/worker').create(process, prefixes);
var ws = opts.worker = opts.worker || require('./process/worker').create(process);
var msgPrefix = 'cluster-rpc.' + opts.name;
var rpcPrefix = msgPrefix + '.rpc';
var resultPrefix = msgPrefix + '.result';
var initPrefix = msgPrefix + '.init';
ws.___listeners = [];
@ -27,7 +30,7 @@ module.exports.create = function (opts) {
args: args
, func: fname
, id: id
, type: prefixes.rpc
, type: rpcPrefix
, _token: token
});
@ -93,21 +96,24 @@ module.exports.create = function (opts) {
return new PromiseA(function (resolve) {
ws.on('message', function (cmd) {
if (opts.debug) { console.log('[cluster-rpc] [worker] message received', cmd); }
if (0 !== (cmd.type||'').indexOf(prefixes.root)) {
//console.log('debug m: mesage', cmd);
if (0 !== (cmd.type||'').indexOf(msgPrefix)) {
//console.log('debug m: ignore msg', cmd);
//console.log(cmd.type, msgPrefix);
//console.log(cmd.type.indexOf(msgPrefix));
return;
}
if (token && token !== cmd._token) {
//console.log('debug m: ignore msg', cmd);
return;
}
switch (cmd.type) {
case prefixes.init:
case initPrefix:
onInit(cmd);
resolve(inst);
break;
case prefixes.result:
case resultPrefix:
onResult(cmd);
break;
default: