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.

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

Improve startup performance #23982

Closed
wezell opened this issue Feb 1, 2023 · 3 comments · Fixed by #24029
Closed

Improve startup performance #23982

wezell opened this issue Feb 1, 2023 · 3 comments · Fixed by #24029

Comments

@wezell
Copy link
Contributor

wezell commented Feb 1, 2023

User Story

Starting a dotCMS instance against a large scale environment can take a long time. One reason for this is that we need to load vanityUrls before we can serve content. If a customer has a lot of sites and a lot of VanityUrls, we have seen it take up to 8 minutes for dotCMS to start up and load.

Acceptance Criteria

Faster time to start.

Proposed Objective

Cloud Engineering

Proposed Priority

Priority 2 - Important

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

https://dotcms.slack.com/archives/CSHTYUR7H/p1675277411732569

Assumptions & Initiation Needs

No response

Sub-Tasks & Estimates

No response

@nollymar
Copy link
Contributor

nollymar commented Mar 6, 2023

PR: #24029

dsilvam pushed a commit that referenced this issue Mar 6, 2023
* #23982 rewrote vanity query

* #23982 speeding up vanity url loading

* #23982 fixes vanity and UT that locked up large installations

* #23982 updates UT, makes adjusting the mod_date optional

* #23982 sonar

* #23982 sonarqube
@dsilvam dsilvam reopened this Mar 6, 2023
erickgonzalez added a commit that referenced this issue Mar 28, 2023
@erickgonzalez erickgonzalez added the Release : 23.01.2 Included in LTS patch release 23.01.2 label Mar 29, 2023
@dsilvam
Copy link
Contributor

dsilvam commented Mar 30, 2023

Passed Internal QA: Loading vanityURLs fast now using provided test dataset.

@josemejias11
Copy link

Approved QA - Tested with the provided dataset

erickgonzalez added a commit that referenced this issue May 5, 2023
@erickgonzalez erickgonzalez added Release : 22.03.6 Included in LTS patch release 22.03.6 and removed Next LTS Release labels May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants