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

DB Suchindex unvollständig bei schrittweiser Indexierung #208

Closed
dtpop opened this issue Jul 24, 2019 · 6 comments
Closed

DB Suchindex unvollständig bei schrittweiser Indexierung #208

dtpop opened this issue Jul 24, 2019 · 6 comments
Labels

Comments

@dtpop
Copy link
Member

dtpop commented Jul 24, 2019

In einer mittleren Präsenz muss der Index schrittweise erstellt werden. Dabei fiel auf, dass der db Suchindex unvollständig ist.
Wenn ich nun aber alle Slices und Artikel aus der Präsenz lösche und den Index auf einen Schlag erstellen kann, ist der Index vollständig.
Ich habe mal ein bisschen im Code gestochert. Mir kommt es so vor, als lösche die Zeile 419 in der search_it.php bei jedem 100er Block die zuvor indexierten Datensätze. Wenn ich testweise diese Zeile auskommentiere, dann ist der Suchindex auch bei schrittweiser Indexierung vollständig.
Allerdings habe ich die Vermutung, dass in diesem wirklich großartigen AddOn diese Zeile doch irgendwie ihren Sinn hat. Daher verzichte ich heute mal auf einen Pullrequest und hoffe, dass Du @tyrant eine Lösung aus dem Hut zaubern kannst und sage dafür Danke!

@dtpop dtpop added the bug label Jul 24, 2019
@tyrant88
Copy link
Member

Man! Super schön formuliert. Ich danke dir.
Ich muss auch immer wieder betonen ich habe das AddOn nur portiert.
Ich bin ziemlich unter Zeitdruck gerade, könntest du versuchen die Zeile 419 so zu ersetzen und das testen?
if ( $_start === false || $_start == 0 ) { $delete->delete(); }

@alxndr-w
Copy link
Member

Vielleicht kann @xong sich erinnern?

@alxndr-w
Copy link
Member

Könnte auch Ursache für #207 sein, jetzt bin ich mir nicht mehr sicher, ob ich vollständig oder schrittweise indexiert habe.

@tyrant88
Copy link
Member

xong kann sich nicht erinnern, weil ich den Teil geändert habe.

tyrant88 added a commit to tyrant88/search_it that referenced this issue Jul 24, 2019
@dtpop
Copy link
Member Author

dtpop commented Jul 31, 2019

Super Norbert. Danke. Habe das getestet und es funktioniert.

@tyrant88
Copy link
Member

cool, danke auch dir.

@tyrant88 tyrant88 mentioned this issue Aug 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants