Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Done ✅ #60

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Done ✅ #60

wants to merge 4 commits into from

Conversation

AbdulRahimOM
Copy link

* Judicially used go routines for fetching data from external API
* Data is stored in map(simple cache) after fetching from external API. (->Good to add expiry here, but haven't implemented as of now)
* Used sync.RWMutex for handling concurrent read and write to this cache
* Pprof endpoints for monitoring
* Periodic logging of statistics (goroutine counts) (for simple monitoring)
* Checking for final person's existence to avoid infinite run
* Predefined number of workers for fetching data
* Proper closing of channels and goroutines
* Context for cancellation, to terminate goroutines and close channels whenever required seperation is found
* Simply ignoring the error if the external API call returns an error, logged at debug level
* Used env variables for configuration
* Net/http used for pprof endpoints, as fiber uses fasthttp
* Rate limiting 

* Judicially used go routines for fetching data from external API
* Data is stored in map(simple cache) after fetching from external API. But need to provide expiry
* Used sync.RWMutex for handling concurrent read and write to this cache
* Pprof endpoints for monitoring
* Periodic logging of statistics (goroutine counts) (for simple monitoring)
* Checking for final person's existence to avoid infinite run
* Predefined number of workers for fetching data
* Proper closing of channels and goroutines
* Context for cancellation, to terminate goroutines and close channels whenever required seperation is found
* Simply ignoring the error if the external API call returns an error, logged at debug level
* Used env variables for configuration
* Net/http used for pprof endpoints, as fiber uses fasthttp
@AbdulRahimOM
Copy link
Author

Abdul Rahim O M
abdulrahimom.work@gmail.com
AbdulRahimOM_CV.pdf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant