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

Ensure index clean up runs before index creation #6634

Merged
merged 1 commit into from Oct 23, 2019
Merged

Conversation

mpfz0r
Copy link
Member

@mpfz0r mpfz0r commented Oct 22, 2019

Move the code from the migration to the constructor.
This avoids a race between our asynchronously executed migration and the
DBProcessingStatusService

Fixes #6383

Move the code from the migration to the constructor.
This avoids a race between our asynchronously executed migration and the
DBProcessingStatusService.

Fixes #6383
@mpfz0r mpfz0r added this to the 3.1.3 milestone Oct 22, 2019
@mpfz0r mpfz0r requested a review from bernd October 22, 2019 15:10
@bernd bernd self-assigned this Oct 23, 2019
@bernd bernd merged commit 6fbf78f into master Oct 23, 2019
@bernd bernd deleted the issue-6383-ter branch October 23, 2019 14:46
bernd pushed a commit that referenced this pull request Oct 23, 2019
Move the code from the migration to the constructor.
This avoids a race between our asynchronously executed migration and the
DBProcessingStatusService.

Fixes #6383

(cherry picked from commit 6fbf78f)
mpfz0r pushed a commit that referenced this pull request Oct 24, 2019
Move the code from the migration to the constructor.
This avoids a race between our asynchronously executed migration and the
DBProcessingStatusService.

Fixes #6383

(cherry picked from commit 6fbf78f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Server start fails with Index with name: compound_0 already exists with a different name
2 participants