Skip to content

feat(Cache): Added cache and reset cache every day#125

Merged
rabgonzalez merged 2 commits into
mainfrom
119-add-caching-on-grahps-endpoints
Apr 24, 2025
Merged

feat(Cache): Added cache and reset cache every day#125
rabgonzalez merged 2 commits into
mainfrom
119-add-caching-on-grahps-endpoints

Conversation

@JonayKB
Copy link
Copy Markdown
Owner

@JonayKB JonayKB commented Apr 24, 2025

No description provided.

@JonayKB JonayKB added the enhancement New feature or request label Apr 24, 2025
@JonayKB JonayKB requested review from Copilot and rabgonzalez April 24, 2025 14:07
@JonayKB JonayKB self-assigned this Apr 24, 2025
@JonayKB JonayKB linked an issue Apr 24, 2025 that may be closed by this pull request
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds caching functionality and a scheduled cache reset to improve performance and data consistency. Key changes include:

  • Introducing CacheResetTask to reset caches daily.
  • Adding @Cacheable annotations to GraphController endpoints.
  • Enabling caching via the new CacheConfig file.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
CacheResetTask.java Implements a daily cache reset task using a scheduled method.
GraphController.java Adds caching to several endpoints via @Cacheable annotations.
CacheConfig.java Configures the application to enable caching.
Comments suppressed due to low confidence (1)

API/src/main/java/es/iespuertodelacruz/routinefights/shared/tasks/CacheResetTask.java:14

  • Consider declaring the logger as a private static final field (e.g., 'private static final Logger LOGGER = ...') to adhere to conventional Java logging practices.
Logger logger = Logger.getLogger(CacheResetTask.class.getName());

@rabgonzalez rabgonzalez merged commit 6898817 into main Apr 24, 2025
3 checks passed
JonayKB pushed a commit that referenced this pull request Jan 24, 2026
feat(Cache): Added cache and reset cache every day
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add caching on grahps endpoints

3 participants