Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

Add 15s cache for ratings #1

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

Conversation

andrewjjenkins
Copy link
Contributor

No description provided.

Andrew Jenkins and others added 3 commits June 20, 2018 15:46
Problem: I just added a webcache to enhance the performance of the
ratings service.  I want to test it but I don't yet want to deploy it to
production, and I don't want to replicate the existing ratings service
in my test environment.  Besides, whenever I run ratings locally, it's
pretty fast - seems to only get slow in production.

Solution: Add a new experiment template for this - I'll define an
experimental deployment of the services I'm interested in and then use
AspenMesh & Istio to route traffic to my deployments.
Problem: Sometimes the ratings service is slow to return results.

Solution: Make a simple cache - if it is less than 15s since we last got
the ratings for this productid, just use the ratings from the cache.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant