Remove redundant lockings if possible

hyperioxx / frontman

Frontman is an open-source API gateway written in Go that allows you to manage your microservices and expose them as a single API endpoint. It acts as a reverse proxy and handles requests from clients, routing them to the appropriate backend service.

https://frontman-labs.github.io/frontman/

GNU General Public License v3.0

5 stars 0 forks source link

Remove redundant lockings if possible #82

Open amityahav opened 1 year ago

amityahav commented 1 year ago

see if we could remove redundant sync lockings for better performance without creating data racing so this needs to be checked and could be with race detection go supports

hyperioxx commented 1 year ago

@amityahav Tried switching to use a sync.Map instead but i feel like i got worse performance from doing so. Will test again

amityahav commented 1 year ago

@hyperioxx as for the clients map I think we can get rid of it's locks as suggested in the other issue https://github.com/Frontman-Labs/frontman/pull/97#pullrequestreview-1376156870