Skip to content

fix(reindex): Fix lock treated as stale on reindexing#911

Merged
viacheslavkol merged 3 commits intomasterfrom
MSEARCH-1181
Mar 3, 2026
Merged

fix(reindex): Fix lock treated as stale on reindexing#911
viacheslavkol merged 3 commits intomasterfrom
MSEARCH-1181

Conversation

@viacheslavkol
Copy link
Contributor

@viacheslavkol viacheslavkol commented Feb 27, 2026

Purpose

Fix lock treated as stale on reindexing

Approach

Add reindex status check before stale lock release logic for a background job

Changes Checklist

  • API Changes: Document any API paths, methods, request or response bodies changed, added, or removed.
  • Database Schema Changes: Indicate any database schema changes and their impact. Confirm that migration scripts were created.
  • Interface Version Changes: Indicate any changes to interface versions.
  • Interface Dependencies: Document added or removed dependencies.
  • Permissions: Document any changes to permissions.
  • Logging: Confirm that logging is appropriately handled.
  • Unit Testing: Confirm that changed classes were covered by unit tests.
  • Integration Testing: Confirm that changed logic was covered by integration tests.
  • Manual Testing: Confirm that changes were tested on local or dev environment.
  • NEWS: Confirm that the NEWS file is updated with relevant information about the changes made in this pull request.

Related Issues

MSEARCH-1181

@viacheslavkol viacheslavkol self-assigned this Feb 27, 2026
@viacheslavkol viacheslavkol marked this pull request as ready for review March 3, 2026 10:33
@viacheslavkol viacheslavkol requested a review from a team as a code owner March 3, 2026 10:33
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@viacheslavkol viacheslavkol merged commit 4472eed into master Mar 3, 2026
16 checks passed
@viacheslavkol viacheslavkol deleted the MSEARCH-1181 branch March 3, 2026 10:54
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.

3 participants