github/node-v0.x-archive
1
1
Fork 0
mirror of https://github.com/nodejs/node-v0.x-archive.git synced 2026-04-28 03:01:10 -04:00
Code Issues Packages Projects Releases Wiki Activity

cluster: fix O(n*m) scan of cmd string

Browse Source 
Don't scan the whole string for a "NODE_CLUSTER_" substring, just check
that the string starts with the expected prefix. The linear scan was
causing a noticeable (but unsurprising) slowdown on messages with a
large .cmd string property.
This commit is contained in:
Ben Noordhuis
2013-04-11 13:18:02 +02:00
parent cd96f0aba8
commit dbbfbe74ca
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/cluster.js
View File

@@ -141,9 +141,10 @@ cluster.setupMaster = function(options) {
// Check if a message is internal only // Check if a message is internal only
var INTERNAL_PREFIX = 'NODE_CLUSTER_'; var INTERNAL_PREFIX = 'NODE_CLUSTER_';
function isInternalMessage(message) { function isInternalMessage(message) {
return (isObject(message) && return isObject(message) &&
typeof message.cmd === 'string' && typeof message.cmd === 'string' &&
message.cmd.indexOf(INTERNAL_PREFIX) === 0); message.cmd.length > INTERNAL_PREFIX.length &&
message.cmd.slice(0, INTERNAL_PREFIX.length) === INTERNAL_PREFIX;
} }
// Modify message object to be internal // Modify message object to be internal