1f038ac1f4
- Existing jobs are moved before start processing them. - Uses a new queue prefix to avoid collisions. - Pub/Sub also changes communication channel. - Job subscriber emits user+host on new jobs. - Batch processor is faulty. See TODO in batch.js.
190 lines
5.5 KiB
JavaScript
190 lines
5.5 KiB
JavaScript
'use strict';
|
|
|
|
var util = require('util');
|
|
var EventEmitter = require('events').EventEmitter;
|
|
var debug = require('./util/debug')('batch');
|
|
var forever = require('./util/forever');
|
|
var queue = require('queue-async');
|
|
var Locker = require('./leader/locker');
|
|
var HostUserQueueMover = require('./maintenance/host-user-queue-mover');
|
|
|
|
function Batch(name, jobSubscriber, jobQueue, jobRunner, jobService, jobPublisher, redisConfig, logger) {
|
|
EventEmitter.call(this);
|
|
this.name = name || 'batch';
|
|
this.jobSubscriber = jobSubscriber;
|
|
this.jobQueue = jobQueue;
|
|
this.jobRunner = jobRunner;
|
|
this.jobService = jobService;
|
|
this.jobPublisher = jobPublisher;
|
|
this.logger = logger;
|
|
this.locker = Locker.create('redis-distlock', { redisConfig: redisConfig });
|
|
this.hostUserQueueMover = new HostUserQueueMover(jobQueue, jobService, this.locker, redisConfig);
|
|
|
|
// map: host => jobId
|
|
this.workingQueues = {};
|
|
}
|
|
util.inherits(Batch, EventEmitter);
|
|
|
|
module.exports = Batch;
|
|
|
|
Batch.prototype.start = function () {
|
|
this.hostUserQueueMover.moveOldJobs(function() {
|
|
this.subscribe();
|
|
}.bind(this));
|
|
};
|
|
|
|
Batch.prototype.subscribe = function () {
|
|
var self = this;
|
|
|
|
this.jobSubscriber.subscribe(
|
|
function onJobHandler(user, host) {
|
|
debug('onJobHandler(%s, %s)', user, host);
|
|
if (self.isProcessingUser(user)) {
|
|
return debug('%s is already processing user=%s', self.name, user);
|
|
}
|
|
|
|
// do forever, it does not throw a stack overflow
|
|
forever(
|
|
function (next) {
|
|
self.locker.lock(host, function(err) {
|
|
// we didn't get the lock for the host
|
|
if (err) {
|
|
debug(
|
|
'Could not lock host=%s for user=%s from %s. Reason: %s',
|
|
host, self.name, user, err.message
|
|
);
|
|
return next(err);
|
|
}
|
|
debug('Locked host=%s for user=%s from %s', host, user, self.name);
|
|
self.processNextJob(user, next);
|
|
});
|
|
},
|
|
function (err) {
|
|
if (err) {
|
|
debug(err.name === 'EmptyQueue' ? err.message : err);
|
|
}
|
|
|
|
self.finishedProcessingUser(user);
|
|
self.locker.unlock(host, debug);
|
|
}
|
|
);
|
|
},
|
|
function onJobSubscriberReady(err) {
|
|
if (err) {
|
|
return self.emit('error', err);
|
|
}
|
|
|
|
self.emit('ready');
|
|
}
|
|
);
|
|
};
|
|
|
|
Batch.prototype.processNextJob = function (user, callback) {
|
|
// This is missing the logic for processing several users within the same host
|
|
// It requires to:
|
|
// - Take care of number of jobs running at the same time per host.
|
|
// - Execute user jobs in order.
|
|
var self = this;
|
|
self.jobQueue.dequeue(user, function (err, jobId) {
|
|
if (err) {
|
|
return callback(err);
|
|
}
|
|
|
|
if (!jobId) {
|
|
var emptyQueueError = new Error('Queue for user="' + user + '" is empty');
|
|
emptyQueueError.name = 'EmptyQueue';
|
|
return callback(emptyQueueError);
|
|
}
|
|
|
|
self.setProcessingJobId(user, jobId);
|
|
|
|
self.jobRunner.run(jobId, function (err, job) {
|
|
self.setProcessingJobId(user, null);
|
|
|
|
if (err) {
|
|
debug(err);
|
|
if (err.name === 'JobNotRunnable') {
|
|
return callback();
|
|
}
|
|
return callback(err);
|
|
}
|
|
|
|
debug('Job=%s status=%s user=%s (failed_reason=%s)', jobId, job.data.status, user, job.failed_reason);
|
|
|
|
self.logger.log(job);
|
|
|
|
self.emit('job:' + job.data.status, jobId);
|
|
|
|
callback();
|
|
});
|
|
});
|
|
};
|
|
|
|
Batch.prototype.drain = function (callback) {
|
|
var self = this;
|
|
var workingUsers = this.getWorkingUsers();
|
|
var batchQueues = queue(workingUsers.length);
|
|
|
|
workingUsers.forEach(function (user) {
|
|
batchQueues.defer(self._drainJob.bind(self), user);
|
|
});
|
|
|
|
batchQueues.awaitAll(function (err) {
|
|
if (err) {
|
|
debug('Something went wrong draining', err);
|
|
} else {
|
|
debug('Drain complete');
|
|
}
|
|
|
|
callback();
|
|
});
|
|
};
|
|
|
|
Batch.prototype._drainJob = function (user, callback) {
|
|
var self = this;
|
|
var job_id = this.getProcessingJobId(user);
|
|
|
|
if (!job_id) {
|
|
return process.nextTick(function () {
|
|
return callback();
|
|
});
|
|
}
|
|
|
|
this.jobService.drain(job_id, function (err) {
|
|
if (err && err.name === 'CancelNotAllowedError') {
|
|
return callback();
|
|
}
|
|
|
|
if (err) {
|
|
return callback(err);
|
|
}
|
|
|
|
self.jobQueue.enqueueFirst(user, job_id, callback);
|
|
});
|
|
};
|
|
|
|
Batch.prototype.stop = function (callback) {
|
|
this.removeAllListeners();
|
|
this.jobSubscriber.unsubscribe(callback);
|
|
};
|
|
|
|
Batch.prototype.isProcessingUser = function(user) {
|
|
return this.workingQueues.hasOwnProperty(user);
|
|
};
|
|
|
|
Batch.prototype.getWorkingUsers = function() {
|
|
return Object.keys(this.workingQueues);
|
|
};
|
|
|
|
Batch.prototype.setProcessingJobId = function(user, jobId) {
|
|
this.workingQueues[user] = jobId;
|
|
};
|
|
|
|
Batch.prototype.getProcessingJobId = function(user) {
|
|
return this.workingQueues[user];
|
|
};
|
|
|
|
Batch.prototype.finishedProcessingUser = function(user) {
|
|
delete this.workingQueues[user];
|
|
};
|