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

Reindexing File Asset causes full reindex. #16545

Closed
mattyarbrough opened this issue May 8, 2019 · 4 comments

Comments

@mattyarbrough
Copy link

commented May 8, 2019

Describe the bug

In 5.1.1 if you choose to just reindex File Asset you get:

Running Contents Index Cache
15:56:18.349 WARN cache.LegacyContentTypeCacheImpl - Structure with inode: ‘File Asset’ not found in db
15:56:18.349 INFO action.ViewCMSMaintenanceAction - Running Contentlet Reindex

and it builds a whole new index

For comparison, choosing just to reindex Blog logs this:

13:32:00.967 INFO action.ViewCMSMaintenanceAction - Running Contentlet Reindex on Structure: Blog
13:32:00.969 INFO action.ViewCMSMaintenanceAction - processAction : Running Contentlet Reindex on structure : Blog
13:32:00.969 INFO util.AdminLogger - class com.dotmarketing.portlets.cmsmaintenance.action.ViewCMSMaintenanceAction : processAction : Running Contentlet Reindex on structure : Blog

And a full new reindex is not triggered.

Steps to reproduce the behavior:

  1. Go to System -> Maintenance -> Index
  2. Click on selector, choose File Asset
  3. Hit reindex
  4. See error

wezell added a commit that referenced this issue May 8, 2019

wezell added a commit that referenced this issue May 8, 2019

@wezell wezell added this to the Rex Current milestone May 9, 2019

@wezell

This comment has been minimized.

Copy link
Contributor

commented May 9, 2019

Found while working on this - If you hit the "delete then reindex" button, the method ContentletIndexAPI.removeContentFromIndexByStructureInode does not work and the content is never removed from the index.

dsilvam added a commit that referenced this issue May 15, 2019

#16545 fixes javascript (#16546)
* #16545 fixes javascript

* #16545 fixing js errors
@dsilvam

This comment has been minimized.

Copy link
Contributor

commented May 15, 2019

PR: #16546

@bryanboza

This comment has been minimized.

Copy link
Contributor

commented May 28, 2019

Fixed, now we are able to reindex content types individually

@bryanboza bryanboza added Passed QA and removed Needs QA labels May 28, 2019

@bryanboza bryanboza moved this from REX to Done in QA May 28, 2019

@wezell wezell closed this May 28, 2019

dsilvam added a commit that referenced this issue May 31, 2019

#16545 fixes javascript (#16546)
* #16545 fixes javascript

* #16545 fixing js errors

@dsilvam dsilvam removed this from Done in QA May 31, 2019

@bryanboza bryanboza added this to REX in QA May 31, 2019

@dsilvam dsilvam self-assigned this May 31, 2019

@bryanboza

This comment has been minimized.

Copy link
Contributor

commented May 31, 2019

Fixed, tested on the release branch and works as expected

@dsilvam dsilvam removed their assignment May 31, 2019

@bryanboza bryanboza moved this from REX to Done in QA Jun 3, 2019

@bryanboza bryanboza added this to Done in Release 5.1.6 Jun 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.