RADAR-base / RADAR-Appserver

General purpose application server for the radar platform currently with capability to schedule push notifications
Apache License 2.0
10 stars 1 forks source link

[FEAT] Cache Github endpoint responses #249

Closed mpgxvii closed 5 months ago

mpgxvii commented 3 years ago

Is your feature request related to a problem? Please describe. Following #247, it would be good to cache Github responses to improve response times, avoid the rate limits, and reduce unnecessary repeated calls.

Describe the solution you'd like As suggested by @blootsvoets and @yatharthranjan, we can use the CachedMap impl and an ExpiringMap implementation for this. Also, taking into account the maximum memory usage.

Describe alternatives you've considered Hazelcast for caching.

Priority 5

Difficulty 6

Additional context Supports #247

yatharthranjan commented 3 years ago

other alternatives:

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

yatharthranjan commented 10 months ago

@mpgxvii is this fixed with #432 ?

mpgxvii commented 5 months ago

Fixed in #432