[MGNLEESOLR-52] Wrong indexing and removing indexes on publication and unpublication Created: 11/Mar/15 Updated: 17/Mar/15 Resolved: 16/Mar/15 |
|
| Status: | Closed |
| Project: | Solr Search Provider |
| Component/s: | None |
| Affects Version/s: | 2.0 |
| Fix Version/s: | 2.1.1 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Richard Gange | Assignee: | Milan Divilek |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | indexing, solr, support | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Template: |
|
||||
| Acceptance criteria: |
Empty
|
||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||
| Description |
|
For publication indexes are sometimes remove when they should not be. This is caused by how activation works. Activation first remove the node and then recreate the node again, but the events in onEvent method are not ordered. Sometimes Event.NODE_ADDED is before Event.Node_REMOVED and in this case we first update index, but then also remove the index. For unpublication indexes are correctly removed, but the exception is logged. This is because property jcr:lockOwner is added to node and then node is removed. So two events are triggered and the indexing fails for Event.PROPERTY_ADDED because node already doesn't exist. 2015-03-11 14:48:23,536 INFO nolia.module.exchangetransactional.XAReceiveFilter: 199:1426099703227 User superuser successfully deactivated /demo-project/members-area on webapp. 2015-03-11 14:48:24,542 ERROR info.magnolia.module.indexer.DataIndexer : Indexer [websiteIndexer]: Repository exception when indexing changes. javax.jcr.ItemNotFoundException: 74b23a3d-957a-45d5-b3bf-9bbab61a9b6b |