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

Invalidate Cache for Reindex #25076

Closed
jdotcms opened this issue May 30, 2023 · 2 comments · Fixed by #25077 or #25165
Closed

Invalidate Cache for Reindex #25076

jdotcms opened this issue May 30, 2023 · 2 comments · Fixed by #25077 or #25165

Comments

@jdotcms
Copy link
Contributor

jdotcms commented May 30, 2023

Parent Issue

No response

Problem Statement

When the policy sent is WAIT_FOR or FORCE; the system has to wait for the ES callback and them proceed to invalidate the cache.

Steps to Reproduce

There are some scenarios where a contentlet is being added,

  • cache invalidated,
  • collection requested before the content is already reindexed
    consequently the previous resulted collection has not the latest results since does not includes the content previously inserted

Acceptance Criteria

When a contentlet is being inserted including WAIT_FOR policy and do a search to a collection that should contain that new contentlet, after the addition, the contentlet should be already there.

dotCMS Version

23.05

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important

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

@fmontes fmontes assigned fmontes and jdotcms and unassigned fmontes May 30, 2023
jdotcms added a commit that referenced this issue May 31, 2023
@rjvelazco rjvelazco self-assigned this Jun 5, 2023
@erickgonzalez erickgonzalez added the OKR : Customer Support Owned by Scott label Jun 5, 2023
@rjvelazco rjvelazco linked a pull request Jun 6, 2023 that will close this issue
2 tasks
fmontes pushed a commit that referenced this issue Jun 6, 2023
* dev: remove retry pages request base on modDate

* fix: tests

* remove
@zJaaal
Copy link
Contributor

zJaaal commented Jun 7, 2023

Passed InternalQA

We need to be sure that everytime we execute a Workflow on Pages Portlet, it only triggers one _search request per Workflow action

Notes for QA (Update)

The table can trigger more _search requests via events, so be sure that the _search we are testing is the one after this request /fire?inode={inode}&indexPolicy=WAIT_FOR. If you don't trigger any table event, we should only see one _search request after we fire the workflow action

Docker Image: [dotcms/dotcms:23.06_c2efd860_SNAPSHOT]

Screenshots

Screen.Recording.2023-06-07.at.10.27.48.AM.mov

@zJaaal zJaaal self-assigned this Jun 7, 2023
@bryanboza
Copy link
Member

Fixed, tested according to the provided video and it is working as expected on release-23.06 // Docker // FF

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment