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

Reasoning for content events initialization #24

Open
karolberezicki opened this issue Mar 19, 2023 · 3 comments
Open

Reasoning for content events initialization #24

karolberezicki opened this issue Mar 19, 2023 · 3 comments

Comments

@karolberezicki
Copy link
Collaborator

Hi,

The content events are currently initialized in IndexingJobService in the method InitContentEvent after the IndexAllOnce.
However, once the application is restarted / redeployed, the content events will be lost unless we'll reindex the whole site again, and for bigger sites it may take couple of hours.
I'm wondering if there's any particular reason why the content events are not initialized during the application startup, but rather in the reindexing job?

@lunchin
Copy link

lunchin commented Mar 23, 2023

Not sure why was done like this. We can try to move to application startup and see if works fine.

@karolberezicki
Copy link
Collaborator Author

I'll try it and create another PR if it proves to be working good.

@ellinge
Copy link

ellinge commented Sep 17, 2024

We had our luck reverting these changes (init of events in job and indexing directly in job), more details in #29

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

No branches or pull requests

3 participants