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

EZP-27735 Solr index not updated immediately after change content priority #1312

Conversation

kmadejski
Copy link
Member

JIRA issue: https://jira.ez.no/browse/EZP-27735

Description

First change of content priority was incorrectly indexed in Solr. Instead of new value, default 0 value was indexed.

@kmadejski kmadejski requested a review from andrerom August 2, 2017 08:52
@andrerom andrerom requested a review from adamwojs August 23, 2017 15:49
@andrerom andrerom merged commit f3fc14e into ezsystems:master Aug 24, 2017
@pkamps
Copy link
Contributor

pkamps commented Aug 27, 2017

The code changes are not affecting how solr gets update. The title is misleading.

The code changes clear the content cache for all objects with new priority (function updatePriority). It's probably a good idea to clear the content cache for those nodes. For example if the node priority is listed on the full view of a node.

@kmadejski
Copy link
Member Author

@pkamps thank you for your feedback. You are partially right, but PR title was created based on JIRA issue. The issue was discovered in relation with Solr and concerned this unwanted behavior which affected only Solr search.

@andrerom
Copy link
Contributor

Title reflect the issue in hybrid setup, where this solves an issue where solr index is not updated since persistence cache is not cleared before the indexing, this was the context where it was reported. Your suggestion for title @pkamps is right, but mainly makes sense in a pure legacy context.

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

Successfully merging this pull request may close these issues.

4 participants