Flagsmith / edge-proxy

The Flagsmith Edge Proxy Docker Image
3 stars 3 forks source link

Modernize codebase to be fully async #77

Closed goncalossilva closed 1 year ago

goncalossilva commented 1 year ago

Continuation of the performance work started in #75 and continued on #76.

This time, making the application fully compatible with async. This doesn't lead to better performance, but it does lead to better consistency. While before we were seeing the maximum CPU utilization jumping up and down, it now stays mostly flat, in parallel with average CPU utilization. This improves predictability our ability to detect problems and situations of under/over-provisioning.

goncalossilva commented 1 year ago

@gagantrivedi perhaps I could get your feedback here? 🙏

goncalossilva commented 1 year ago

Hey! Is there anything preventing this merge? We could do away with our internal fork once it is. 😊