-
Notifications
You must be signed in to change notification settings - Fork 54
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
Job queue #257
Job queue #257
Conversation
@blond, использование этого расшареного пула в технологиях |
var promise = vow.promise(); | ||
var args = [].slice.call(arguments, 1); | ||
|
||
this._workers(path, args, function (res) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А два аргумента сюда вернуть не получится? В ноде принято в таких случаях использовать сигнатуру callback(error, result)
@j0tunn Нужна документация о том, как этим пользоваться в технологиях |
var res = null; | ||
var err = null; | ||
try { | ||
res = require(module).apply(null, args); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Получается, что функция может работать только синхронно?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
да
🆙 |
@@ -1432,3 +1432,28 @@ Cache Node::getNodeCache(String targetName) | |||
|
|||
* Валидация кэша: [Технология deps](/mdevils/enb/blob/master/techs/deps.js#L33) | |||
* Кэширование результатов сборки: [Технология deps](/mdevils/enb/blob/master/techs/deps.js#L73) | |||
|
|||
node.getSharedResources |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Может фунуции/методы указывать с сигнатурой?
node.getSharedResources()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А, там везде так
Можно ли написать тесты на это поведение? |
Теоретически можно, но это просто обертка над модулем
Тупо time'ом. Прогнал без использования очереди - сборка заняла
Кол-во воркеров на скорость влияют. Чем меньше воркеров - тем меньше накладных расходов на их создание, но при этом меньше тасок, которые могут выполняться одновременно. Оптимальный вариант - по количеству ядер в системе, как оно по дефолту и есть. Сегодня планирую поиграться с |
Понятия не имею как |
Поигрался с настройками |
Shared Job Queue, which can be used in technologies.