Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Privacy
    • Imprint
    • Contact
    • Login methods
  • Sign in
  • ARSnova Backend ARSnova Backend
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 27
    • Issues 27
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ARSnovaARSnova
  • ARSnova BackendARSnova Backend
  • Merge requests
  • !2
The source project of this merge request has been removed.

Caching fixes

Merged Daniel Gerhardt requested to merge (removed):caching-fixes into master May 25, 2015
  • Overview 0
  • Pipelines 0
  • Changes

This PR fixes some caching bugs:

  • Cached methods have been called locally inside of CouchDBDao. The caching proxy was bypassed in these cases. All public methods (even those not involved in caching) are now called on the proxy instance. The here provided implementation is not optimal since it goes against SoC but can serve as a temporary fix. A better solution might involve AspectJ's weaving.

  • A wrong key for cache eviction has been used for NewAnswerEvent in CacheBuster.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: caching-fixes