Skip to content

signal to the webapp once partial reindex is done #1187

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.

Sign up for GitHub

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

Jump to bottom
Closed
vladak opened this issue Sep 16, 2016 · 3 comments · Fixed by #1423
Closed

signal to the webapp once partial reindex is done #1187

vladak opened this issue Sep 16, 2016 · 3 comments · Fixed by #1423
Assignees
@vladak
Labels
enhancement
Milestone
1.0

Comments

@vladak
Copy link
Member

vladak commented Sep 16, 2016 •
edited
Loading

Once the configuration listener protocol is extended so that allows more than just Configuration object to be passed, the protocol should be further extended so that partial reindex notifies the webapp to call maybeRefresh() on specific SearcherManager objects. This is a follow-up to the changes for #1132.
@vladak vladak mentioned this issue Sep 16, 2016
Merged
@tarzanek
Copy link
Contributor

@tulinkry I think you have this ready with your messaging addition to listener, no ?

@tulinkry
Copy link
Contributor

tulinkry commented Sep 23, 2016 •
edited
Loading

Yes. Not ready but on the way

@tulinkry tulinkry mentioned this issue Sep 27, 2016
Merged
@vladak vladak added this to the 0.13 milestone Oct 7, 2016
@vladak
Copy link
Member Author

vladak commented Oct 7, 2016

Also, with this change it would be good to relax the IndexRefreshPeriod which is currently set to 1 minute so every minute the webapp tries to reopen the indexes which causes syscall and I/O churn.

@vladak vladak mentioned this issue Feb 7, 2017
Closed
@vladak vladak self-assigned this Feb 16, 2017
@vladak vladak mentioned this issue Feb 17, 2017
Merged
vladak added a commit to vladak/OpenGrok that referenced this issue Feb 17, 2017
@vladak
refresh SearcherManagers after partial reindex
2ba0cd1 
fixes oracle#1187
vladak pushed a commit that referenced this issue Feb 20, 2017
refresh SearcherManagers after partial reindex (#1423)
7056f12 
* refresh SearcherManagers after partial reindex
fixes #1187

* override applyMessage

* add validation

* document
ozankiraz pushed a commit to ozankiraz/OpenGrok that referenced this issue Feb 28, 2017
@ozankiraz
refresh SearcherManagers after partial reindex (oracle#1423)
7fd8a01 
* refresh SearcherManagers after partial reindex
fixes oracle#1187

* override applyMessage

* add validation

* document
@vladak vladak mentioned this issue Aug 11, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vladak vladak

Labels
enhancement
Projects
None yet
Milestone
1.0
Development

Successfully merging a pull request may close this issue.

refresh SearcherManagers after partial reindex vladak/OpenGrok
3 participants
@tarzanek @vladak @tulinkry