Reputation: 1973
Is there a technique to break an async loop, if it do not completes within an expected time period? I have a code like this:
(async()=>{
for await(let t of asynDataStreamOrGenerator){
//some data processing
}
//some other code I need to run based on whatever data collected by
//asyncDataStreamOrGenerators within given time period
})()
If this loop is not completed within a timespan, break out of the loop and process the request further.
Upvotes: 4
Views: 2987
Reputation: 10591
You can use a timeout Promise
(timer
in the code)
and use Promise.race
on each iteration.
The code below would print up to around 30 while the generator can generate more.
async function wait(ms) {
return new Promise(r=>setTimeout(r, ms))
}
async function* asynDataStreamOrGenerator() {
for (let i = 0; i < 100; ++i) {
await wait(30)
yield i;
}
}
async function* iterate_until(generator, timeout) {
let timer = wait(timeout).then(_=>Promise.reject("TimeOut"))
for (;;) {
let it = generator.next()
let result = await Promise.race([timer, it])
if (result.done) break;
yield result.value;
}
}
{(async () => {
try {
for await (let t of iterate_until(asynDataStreamOrGenerator(), 1000)) {
console.log(t)
}
} catch (e) { /* catch timeout, rethrow if needed*/ }
})()}
Upvotes: 3
Reputation: 1074276
(See also the community wiki answer I posted with an alternative approach.)
In a comment you've said:
I am designing a consensus algorithm, where every source needs to send the response within a given time frame. If some of such participants are dead!, I mean they do not send values, the loop will be held for ever!
That sounds like a timeout to me. The usual way to implement a timeout is via Promise.race
with a promise wrapped around a timer mechanism (setTimeout
or similar). Promise.race
watches the promises you pass into it and settles as soon as any of them settles (passing on that fulfillment or rejection), disregarding how any of the others settle later.
To do that, you'll need to loop another way instead of for-await-of
and use the promise of the result object directly rather than indirectly. Let's say you have a utility function:
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
That returns a promise it fulfills X milliseconds later with whatever value you provide (if any).
Then:
(async () => {
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
const it = asynDataStreamOrGenerator[Symbol.asyncIterator]();
try {
while (true) {
const p = it.next();
const result = await Promise.race([p, delay(TIMEOUT, GOT_TIMEOUT)]);
if (result === GOT_TIMEOUT) {
// Didn't get a response in time
console.log("Timeout");
} else {
// Got a response
if (result.done) {
// Iteration complete
console.log("Iteration complete");
break;
}
// ...some data processing on `result.value`...
console.log(`Process ${result.value}`);
}
}
} finally {
try {
it.return?.(); // Close the iterator if it needs closing
} catch { }
}
})();
Live Example using random durations for the async iterator's work, but forcing a timeout on the third iteration:
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
async function* example() {
for (let i = 1; i <= 6; ++i) {
const ms = i === 3 ? 600 : Math.floor(Math.random() * 100);
await delay(ms);
yield i;
}
}
(async () => {
const asynDataStreamOrGenerator = example();
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
const it = asynDataStreamOrGenerator[Symbol.asyncIterator]();
try {
while (true) {
const p = it.next();
const start = Date.now();
const result = await Promise.race([p, delay(TIMEOUT, GOT_TIMEOUT)]);
const elapsed = Date.now() - start;
if (result === GOT_TIMEOUT) {
// Didn't get a response in time
console.log(`Got timeout in ${elapsed}ms`);
} else {
// Got a response
if (result.done) {
// Iteration complete
console.log(`Got iteration complete result in ${elapsed}ms`);
break;
}
// ...some data processing on `result.value`...
console.log(`Got result ${result.value} to process in ${elapsed}ms`);
}
}
} finally {
try {
it.return?.(); // Close the iterator if it needs closing
} catch { }
}
})();
.as-console-wrapper {
max-height: 100% !important;
}
Here's that example with the timeout on the first iteration, since you seemed concerned about that case:
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
async function* example() {
for (let i = 1; i <= 6; ++i) {
const ms = i === 1 ? 600 : Math.floor(Math.random() * 100);
await delay(ms);
yield i;
}
}
(async () => {
const asynDataStreamOrGenerator = example();
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
const it = asynDataStreamOrGenerator[Symbol.asyncIterator]();
try {
while (true) {
const p = it.next();
const start = Date.now();
const result = await Promise.race([p, delay(TIMEOUT, GOT_TIMEOUT)]);
const elapsed = Date.now() - start;
if (result === GOT_TIMEOUT) {
// Didn't get a response in time
console.log(`Got timeout in ${elapsed}ms`);
} else {
// Got a response
if (result.done) {
// Iteration complete
console.log(`Got iteration complete result in ${elapsed}ms`);
break;
}
// ...some data processing on `result.value`...
console.log(`Got result ${result.value} to process in ${elapsed}ms`);
}
}
} finally {
try {
it.return?.(); // Close the iterator if it needs closing
} catch { }
}
})();
.as-console-wrapper {
max-height: 100% !important;
}
If you don't want the processing to hold up collection of the next value, you could not await
the processing that you do (perhaps build up an array of the promises for completion of that processing and Promise.all
them at the end).
Or if you want to bail out of the entire operation:
(async () => {
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
const results = [];
const it = asynDataStreamOrGenerator[Symbol.asyncIterator]();
try {
while (true) {
const p = it.next();
const result = await Promise.race([p, delay(TIMEOUT, GOT_TIMEOUT)]);
if (result === GOT_TIMEOUT) {
// Didn't get a response in time, bail
console.log("Timeout");
break;
}
// Got a response
if (result.done) {
// Iteration complete
console.log("Iteration complete");
break;
}
console.log(`Got ${result.value}`);
results.push(result.value);
}
} finally {
try {
it.return?.();
} catch { }
}
// ...code here to process the contents of `results`...
for (const value of results) {
console.log(`Process ${value}`);
}
})();
Live Example:
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
async function* example() {
for (let i = 1; i <= 6; ++i) {
const ms = i === 3 ? 600 : Math.floor(Math.random() * 100);
await delay(ms);
yield i;
}
}
(async () => {
const asynDataStreamOrGenerator = example(); // For the example
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
const results = [];
const it = asynDataStreamOrGenerator[Symbol.asyncIterator]();
try {
while (true) {
const p = it.next();
const start = Date.now();
const result = await Promise.race([p, delay(TIMEOUT, GOT_TIMEOUT)]);
const elapsed = Date.now() - start;
if (result === GOT_TIMEOUT) {
// Didn't get a response in time, bail
console.log(`Got timeout after ${elapsed}ms`);
break;
}
// Got a response
if (result.done) {
// Iteration complete
console.log(`Got iteration complete after ${elapsed}ms`);
break;
}
console.log(`Got value ${result.value} after ${elapsed}ms`);
results.push(result.value);
}
} finally {
try {
it.return?.();
} catch { }
}
// ...code here to process the contents of `results`...
for (const value of results) {
console.log(`Process ${value}`);
}
})();
.as-console-wrapper {
max-height: 100% !important;
}
And again where it times out on the first pass but not every pass (since this bails on the first timeout, we don't see subsequent ones):
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
async function* example() {
for (let i = 1; i <= 6; ++i) {
const ms = i === 1 ? 600 : Math.floor(Math.random() * 100);
await delay(ms);
yield i;
}
}
(async () => {
const asynDataStreamOrGenerator = example(); // For the example
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
const results = [];
const it = asynDataStreamOrGenerator[Symbol.asyncIterator]();
try {
while (true) {
const p = it.next();
const start = Date.now();
const result = await Promise.race([p, delay(TIMEOUT, GOT_TIMEOUT)]);
const elapsed = Date.now() - start;
if (result === GOT_TIMEOUT) {
// Didn't get a response in time, bail
console.log(`Got timeout after ${elapsed}ms`);
break;
}
// Got a response
if (result.done) {
// Iteration complete
console.log(`Got iteration complete after ${elapsed}ms`);
break;
}
console.log(`Got value ${result.value} after ${elapsed}ms`);
results.push(result.value);
}
} finally {
try {
it.return?.();
} catch { }
}
// ...code here to process the contents of `results`...
for (const value of results) {
console.log(`Process ${value}`);
}
})();
.as-console-wrapper {
max-height: 100% !important;
}
Or some combination of the two. You'll need to tweak this based on what you're really doing, but that's a direction that seems reasonable.
In all of the above:
it.return?.();
with if (it.return) { it.return(); }
if your environment doesn't support optional chaining yet.catch { }
with catch (e) { }
if your environment doesn't support optional catch
bindings yet.Upvotes: 3
Reputation: 1074276
In the comments on the question, Heretic Monkey suggested wrapping the async iterable in another one that implements a timeout. That's a very good idea, because then the code using that wrapper can use for-await-of
.
If you want to keep going after a timeout, it looks something like this:
async function* timeoutWrapper(asyncIterable, timeoutDuration, timeoutValue) {
const it = asyncIterable[Symbol.asyncIterator]();
try {
while (true) {
const result = await Promise.race([
it.next(),
delay(timeoutDuration, timeoutValue)
]);
if (result === timeoutValue) {
yield timeoutValue;
} else if (result.done) {
break;
} else {
yield result.value;
}
}
} finally {
it.return?.();
}
}
Using it:
for await (const t of timeoutWrapper(asynDataStreamOrGenerator, TIMEOUT, GOT_TIMEOUT)) {
if (t === GOT_TIMEOUT) {
// Didn't get a response in time
console.log("Timeout");
} else {
// Got a response
// ...some data processing on `result.value`...
console.log(`Process ${t}`);
}
}
console.log("Done");
Live Example:
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
async function* example() {
for (let i = 1; i <= 6; ++i) {
await delay(200 + i * 100);
yield i;
}
}
async function* timeoutWrapper(asyncIterable, timeoutDuration, timeoutValue) {
const it = asyncIterable[Symbol.asyncIterator]();
try {
while (true) {
const result = await Promise.race([
it.next(),
delay(timeoutDuration, timeoutValue)
]);
if (result === timeoutValue) {
yield timeoutValue;
} else if (result.done) {
break;
} else {
yield result.value;
}
}
} finally {
it.return?.();
}
}
(async () => {
const asynDataStreamOrGenerator = example();
const TIMEOUT = 500; // Milliseconds
const GOT_TIMEOUT = {};
for await (const t of timeoutWrapper(asynDataStreamOrGenerator, TIMEOUT, GOT_TIMEOUT)) {
if (t === GOT_TIMEOUT) {
// Didn't get a response in time
console.log("Timeout");
} else {
// Got a response
// ...some data processing on `result.value`...
console.log(`Process ${t}`);
}
}
console.log("Done");
})();
If you don't want to keep going after a timeout, you could have it throw an error (terminating the iteration) instead:
async function* timeoutWrapper(asyncIterable, timeoutDuration) {
const TIMEOUT_VALUE = {};
const it = asyncIterable[Symbol.asyncIterator]();
try {
while (true) {
const result = await Promise.race([
it.next(),
delay(timeoutDuration, TIMEOUT_VALUE)
]);
if (result === TIMEOUT_VALUE) {
throw new Error(`Timeout after ${timeoutDuration}ms`);
} else if (result.done) {
break;
} else {
yield result.value;
}
}
} finally {
it.return?.();
}
}
Using it:
try {
for await (const t of timeoutWrapper(asynDataStreamOrGenerator, TIMEOUT)) {
// Got a response
console.log(`Process ${t}`);
}
console.log("Done");
} catch (e) {
console.error(e.message);
}
Live Example:
const delay = (ms, value) => new Promise(resolve => {
setTimeout(resolve, ms, value);
});
async function* example() {
for (let i = 1; i <= 6; ++i) {
await delay(200 + i * 100);
yield i;
}
}
async function* timeoutWrapper(asyncIterable, timeoutDuration) {
const TIMEOUT_VALUE = {};
const it = asyncIterable[Symbol.asyncIterator]();
try {
while (true) {
const result = await Promise.race([
it.next(),
delay(timeoutDuration, TIMEOUT_VALUE)
]);
if (result === TIMEOUT_VALUE) {
throw new Error(`Timeout after ${timeoutDuration}ms`);
} else if (result.done) {
break;
} else {
yield result.value;
}
}
} finally {
it.return?.();
}
}
(async () => {
const asynDataStreamOrGenerator = example();
const TIMEOUT = 500; // Milliseconds
try {
for await (const t of timeoutWrapper(asynDataStreamOrGenerator, TIMEOUT)) {
// Got a response
console.log(`Process ${t}`);
}
console.log("Done");
} catch (e) {
console.error(e.message);
}
})();
Upvotes: 1