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

avoid cascading indexing #291

Merged
merged 1 commit into from
Sep 1, 2020
Merged

avoid cascading indexing #291

merged 1 commit into from
Sep 1, 2020

Conversation

TobiasKrais
Copy link
Member

Verhindert einen Aufruf des EPs während einer laufenden Indexierung. Sonst könnte eine Indexierung kaskadierend aufgerufen werden.

@TobiasKrais TobiasKrais requested a review from tyrant88 September 1, 2020 15:00
Copy link
Member

@tyrant88 tyrant88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

müsste nicht eigentlich der rex_config::remove() Aufruf an den Anfang des IF-Blockes wandern, also VOR die ganzen Funktions-Aufrufe?

@tyrant88
Copy link
Member

tyrant88 commented Sep 1, 2020

Würde das nicht bereits den Effekt verhindern?

@TobiasKrais
Copy link
Member Author

Ja, würde es. Aber: wenn die Indexierung das Zeitlimit des Servers überschreitet, bleibt die Indexierungsaufgabe unvollendet wenn der Eintrag in der Config entfernt wird. Mit dieser Lösung wird er auf jeden Fall beendet, da die Indexierung später fortgesetzt wird.

@tyrant88
Copy link
Member

tyrant88 commented Sep 1, 2020

Aber dann wird jeder andere parallele Aufruf (der die IF-Abfrage erfüllt) trotzdem eine "Kaskade" auslösen, oder?
D.h. du verhinderst einige zusätzliche Aufrufe, aber nicht alle. Z.B. einfach im Backend navigieren.

@TobiasKrais
Copy link
Member Author

Ja, das lässt sich nicht ganz verhindern. Will man es komplett verhindern, würde man die Möglichkeit verlieren bei zu langer Skriptlaufzeit den Indexierungsprozess wieder aufzunehmen.

Mir geht es vor allem um die Kaskadierung innerhalb eines Indexierungsprozesses. Der hat mir heute meine Webseite für fast eine Stunde unerreichbar gemacht, da es so extrem viele Seitenaufrufe gab.

@tyrant88
Copy link
Member

tyrant88 commented Sep 1, 2020

dann sollten wir nochmal über die timer Lösung nachdenken, die ich im Sinn hatte...
Aber ich drücke schon mal aufs "Knöpfchen"

@tyrant88 tyrant88 merged commit 46bdb27 into master Sep 1, 2020
@delete-merged-branch delete-merged-branch bot deleted the limit branch September 1, 2020 16:53
@tyrant88 tyrant88 mentioned this pull request Oct 13, 2020
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

Successfully merging this pull request may close these issues.

2 participants