search

Xabaril / AspNetCore.Diagnostics.HealthChecks

Enterprise HealthChecks for ASP.NET Core Diagnostics Package
Apache License 2.0
4.05k stars 794 forks source link

Parallel verification and notification #594

Open GustavoFonsecaDeAlmeida opened 4 years ago

GustavoFonsecaDeAlmeida commented 4 years ago

What would you like to be added: During the integrity application process, I saw that the verification and notification processes (if configured) work synchronously.

Verify

Notification

Couldn't these two processes take place using parallel foreach?

I believe that the verification process does not need a specific order so the parallel foreach would suit perfectly.

Why is this needed: In a scenario where a verification process may take longer than necessary for some reason, other subsequent checks would wait unnecessarily long.

The same scenario for notification.

GustavoFonsecaDeAlmeida commented 4 years ago

@unaizorrilla Can we talk about? I can make PR