bielu / bielu.examine.providers

MIT License
1 stars 1 forks source link

Empty indexes are created when Umbraco/IIS restarts #5

Closed mcl-sz closed 1 week ago

mcl-sz commented 4 months ago

To replicate this issue:

After indexing afbeelding After restart and opening the Examine Management dashboard afbeelding

bielu commented 4 months ago

Hey @mcl-sz I moved control over indexes from umbraco to custom service, it seems to be more stable, can you test it in this beta: beta.638456899586285693

mcl-sz commented 4 months ago

Hey @bielu , this works almost fine, when an index already exists CurrentIndexName is not set so the GetDocumentCount does not work properly and is re-indexed

I have made a pull request for this. Not the prettiest solution, but it seems to work https://github.com/bielu/bielu.examine.providers/pull/7

bielu commented 4 months ago

Hey @mcl-sz, I am working on next release for it you can follow pr https://github.com/bielu/bielu.examine.providers/pull/1 I hope get it finished today!

bielu commented 4 months ago

hey @mcl-sz, Latest package 1.0.0-beta.638462976255970049 should resolve this issue, please follow new information docs how to use :) please let me know if I missed something in docs!

bielu commented 3 months ago

hey @mcl-sz did you had chance to test it on your end?

mcl-sz commented 3 months ago

Hi @bielu, indexing seems to work stably now . Thank you very much! Unfortunately, the statistics in the Examine Management dashboard are not updated when an index is recreated. This makes it somewhat difficult to determine whether indexing was successful if you are not using Kibana.

It would be great if you could find a solution for this as well.

bielu commented 3 months ago

@mcl-sz it happens also with examine lucene indexes 😅 so that is good question if that can be fixed

mcl-sz commented 2 weeks ago

Hi @bielu , i've created a pr(https://github.com/bielu/bielu.examine.providers/pull/24) for fixing the statistics in the Examine Management dashboard. Could you give it a try?

I've commited them to the poc/observer branch because the observer patern works best for me. Are you planning to include the observer pattern in the main-branch?

bielu commented 1 week ago

@mcl-sz I merged all changes now to main, I just wanted play with it before merging. I am closing ticket as well as I think all work as should