What do you think about Promise.allSettled()
?
To me, allSettled
looks like a solution in search of a problem. That problem is developers not handling errors.
The Concept
Promise.allSettled()
has a very simple design:
const allSettled = (promises) => Promise.all(promises.map(entry => entry
.then((value) => ({ status: 'fulfilled', value }))
.catch((reason) => ({ status: 'rejected', reason }))
));
It provides a "consistent" outcome object – well, status
is consistent so you can .filter()
more cleanly than using Object.hasOwn()
, but value
and reason
are intentionally different so you can't mix them up.
Mostly, allSettled
adds a .catch()
to each promise for you.
Handle Your Errors
But here's my sticking point: if you are calling a group of services in parallel and you know one or more can fail, yet it doesn't really matter...why aren't you writing error handling for that?
const getFlakyService = (payload) => fetch(flakyUrl, payload);
Promise.allSettled([
getFlakyService({ type: 'reptiles' }),
getFlakyService({ type: 'mammals' }),
getFlakyService({ type: 'birds' }),
getFlakyService({ type: 'amphibians' }),
]).then((outcomes) => outcomes
.filter(({ status }) => status === 'fulfilled'))
});
How much effort are we saving compared to this:
const getFlakyService = (payload) => fetch(flakyUrl, payload)
// We don't really care about the failures
.catch(() => undefined);
Promise.all([
getFlakyService({ type: 'reptiles' }),
getFlakyService({ type: 'mammals' }),
getFlakyService({ type: 'birds' }),
getFlakyService({ type: 'amphibians' }),
]).then((data) => { /* ... */ });
If you care about which calls are failing, you likely need the request information accessible for tracking, which isn't guaranteed to be available in the reason
. Promise.allSettled
is even less helpful in this case and it makes more sense to write your own error handling.
const getFlakyService = (payload) => fetch(flakyUrl, payload)
// Send the failures details to a tracking/logging layer
.catch((error) => trackRequestError(flakyUrl, payload, error);
Promise.all([
getFlakyService({ type: 'reptiles' }),
getFlakyService({ type: 'mammals' }),
getFlakyService({ type: 'birds' }),
getFlakyService({ type: 'amphibians' }),
]).then((data) => { /* ... */ });
I will grant that the standardization of the "outcome" could be convenient. With allSettled
you can count the failures once they all complete. But that's true with custom error handling as well.
Conclusion
I'll continue to use Promise.all()
for the near future, but I'm interested to hear about your use cases for Promise.allSettled()
and why you prefer it.
Top comments (0)