Skip to content

Conversation

@AlternateIf
Copy link
Contributor

@AlternateIf AlternateIf commented Dec 2, 2024

aws opensearch failed me multiple times when reindexing big indices. apparently there is some time of connection time limit set and even passing "timeout" for the reinidex call did not help. When the time limit is reached an exception occurs. (i think it was a 504 gateway timeout)

so to fix this i switched to letting the reindex run asynchronously and only querying the status periodically. this way we get instant responses from the index call and don't run into timeouts.

We can refine this approach. this is just my WIP i used to temporarly fix for a project where that problem occured.

what do you think? @fashxp @mattamon

@github-actions
Copy link

github-actions bot commented Dec 2, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2024

@ghost ghost added the Pimcore:ToDo label Dec 3, 2024
@AlternateIf
Copy link
Contributor Author

AlternateIf commented Dec 3, 2024

@fashxp
Copy link
Member

fashxp commented Apr 18, 2025

@AlternateIf looks good to me. but it seems that in the PR there are some commits from a different user. we need to fix that before we can merge.

@AlternateIf
Copy link
Contributor Author

@fashxp can you point me to the commits from another user? i can't see them.
github shows me only commits from me and github bot

i can just recreate the pr with a new branch if that's easier. just tell me what branch i should use as base :)

@mattamon
Copy link
Contributor

@AlternateIf you committed with e-mail david.magoc@yahoo.de and AlternateIf@users.noreply.github.com.
My guess would be that the first mail is not part of your GitHub account, so you have to add this e-mail to your GitHub account please.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants