Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Cache when getting Versionable info #26931

Closed
erickgonzalez opened this issue Dec 8, 2023 · 0 comments · Fixed by #26968
Closed

Add Cache when getting Versionable info #26931

erickgonzalez opened this issue Dec 8, 2023 · 0 comments · Fixed by #26968
Assignees
@erickgonzalez
Labels
LTS: Excluded Ticket that has been excluded from at least one LTS QA : Not Needed Release : 23.10.24 v1 Included in LTS patch release 23.10.24 v1 Release : 24.01.26 Bug Fixing Team : Bug Fixers Type : Task

Comments

@erickgonzalez
Copy link
Contributor

Parent Issue

No response

Task

Due to the lack of caching, system performance is severely impacted. Implementing a caching mechanism for retrieving Versionable information is imperative.

Proposed Objective

Code Maintenance

Proposed Priority

Priority 2 - Important

Acceptance Criteria

No response

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
@erickgonzalez erickgonzalez self-assigned this Dec 11, 2023
erickgonzalez added a commit that referenced this issue Dec 11, 2023
@erickgonzalez
chore: add cache to contentletVersionInfo ref: #26931
d76f534
erickgonzalez added a commit that referenced this issue Dec 11, 2023
@erickgonzalez
test: add test ref: #26931
846a7cb
@erickgonzalez erickgonzalez linked a pull request Dec 11, 2023 that will close this issue
Add cache to contentversioninfo #26968
Merged
erickgonzalez added a commit that referenced this issue Dec 12, 2023
@erickgonzalez
#26931 include in 23.10.24
389abb4
erickgonzalez added a commit that referenced this issue Dec 12, 2023
@erickgonzalez
#26931 include in 23.10.24
98d4e5e
@erickgonzalez erickgonzalez added the Release : 23.10.24 v1 Included in LTS patch release 23.10.24 v1 label Dec 19, 2023
github-merge-queue bot pushed a commit that referenced this issue Dec 27, 2023
@erickgonzalez @wezell
Add cache to contentversioninfo (#26968)
48caf15 
* chore: add cache to contentletVersionInfo ref: #26931

* test: add test ref: #26931

* feedback

* feedback

---------

Co-authored-by: Will Ezell <will@dotcms.com>
@erickgonzalez erickgonzalez added QA : Not Needed Release : 24.01.26 Bug Fixing LTS: Excluded Ticket that has been excluded from at least one LTS labels Dec 27, 2023
@bryanboza bryanboza added Release : 23.12.21 SEO Improvements and removed Release : 24.01.26 Bug Fixing labels Jan 3, 2024
@erickgonzalez erickgonzalez added Release : 24.01.26 Bug Fixing and removed Release : 23.12.21 SEO Improvements labels Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erickgonzalez erickgonzalez

Labels
LTS: Excluded Ticket that has been excluded from at least one LTS QA : Not Needed Release : 23.10.24 v1 Included in LTS patch release 23.10.24 v1 Release : 24.01.26 Bug Fixing Team : Bug Fixers Type : Task
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add cache to contentversioninfo dotCMS/core
3 participants
@bryanboza @erickgonzalez @damen-dotcms