Cache etag in DifferencesResource
Reported by Ben Hood | October 12th, 2011 @ 12:27 PM | in 0.9-series (closed)
At the moment the etag in the DifferencesResource uses the current sequence id, which causes a DB round trip for each request.
Comments and changes to this ticket
-
Ben Hood October 12th, 2011 @ 12:27 PM
- Tag changed from “caching, etag” to “caching”
-
Ben Hood October 12th, 2011 @ 12:27 PM
- Tag changed from “caching” to “caching, etags”
-
Ben Hood October 12th, 2011 @ 01:55 PM
(from [4667df1c31c18059afdc3e0c01877ac61b8d7595]) [#378] Add simple RT cache for domain sequence ids https://github.com/lshift/diffa/commit/4667df1c31c18059afdc3e0c0187...
-
Ben Hood October 12th, 2011 @ 02:22 PM
(from [59bb95849034503a23985ba856cc8f53ca0b325c]) [#378] Attempt to nullify bad test https://github.com/lshift/diffa/commit/59bb95849034503a23985ba856cc...
-
Ben Hood October 12th, 2011 @ 09:00 PM
- State changed from “new” to “resolved”
- Assigned user changed from “Ben Hood” to “Paul Jones”
PR has been raised, but the patch was merged, so this is one for post QA. On reflection, maybe we should take down a follow up ticket to put more holistic application level caching into the relevant stores so that we are reliant on Hibernate.
-
Paul Jones October 12th, 2011 @ 10:47 PM
- State changed from “resolved” to “closed”
Change looks to be ok to me.
-
Ben Hood November 14th, 2011 @ 05:35 PM
- Milestone set to “0.9-series”
- Milestone order changed from “127” to “0”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
A Real Time Differencing Tool.
People watching this ticket
Ben Hood
Referenced by
-
384
Hand cache hibernate stores
Due to perennial caching issues we have been experiencing...
-
384
Hand cache hibernate stores
This should replace the caching in #367,
#368
and #378.