Skip to content

Conversation

as51340
Copy link
Contributor

@as51340 as51340 commented Mar 20, 2025

The failover in the MT code now has a different logic. We 1st try to find the instance which has most newest databases. If there are multiple instances that are newest on the same number of databases, the sum of timestamps of all databases decides. The instance which has larger sum it is assumed to be newer.

@as51340 as51340 self-assigned this Mar 20, 2025
@as51340 as51340 added CI -build=release -test=core Run release build and core tests on push CI -build=release -test=e2e Run release build and e2e tests on push labels Mar 20, 2025
@as51340 as51340 force-pushed the use-only-latest-ts branch from 2c6ab72 to 1125bae Compare March 25, 2025 08:57
@as51340 as51340 changed the base branch from ha-mt-failover to master March 27, 2025 07:08
@as51340 as51340 mentioned this pull request Mar 27, 2025
@as51340 as51340 added this to the mg-v3.2.0 milestone Mar 27, 2025
@as51340 as51340 marked this pull request as ready for review March 27, 2025 12:51
@as51340 as51340 requested a review from andrejtonev March 27, 2025 12:51
@as51340 as51340 changed the title Use only latest timestamp when doing a failover Improve the failover process in HA Mar 28, 2025
@as51340 as51340 changed the title Improve the failover process in HA Improve the failover process in HA when using multitenancy Mar 28, 2025
@as51340
Copy link
Contributor Author

as51340 commented Mar 31, 2025

Tracking

  • [Link to Epic/Issue]

Standard development

CI Testing Labels

  • Select the appropriate CI test labels (CI -build=build-name -test=test-suite)

Documentation checklist

  • Add the documentation label
  • Add the bug / feature label
  • Add the milestone for which this feature is intended
    • If not known, set for a later milestone
  • Write a release note, including added/changed clauses
    • There is now an improved logic for selecting the new MAIN instance during the failover in the MT environment. Instance which has most newest databases is considered the newest. Users will now have a smaller chance of experiencing the loss of committed data. #2812,
  • Documentation PR link memgraph/documentation
    • Is back linked to this development PR
  • [ Tag someone from docs team ] @katarinasupe
@as51340 as51340 requested a review from andrejtonev April 2, 2025 06:59
@as51340 as51340 force-pushed the use-only-latest-ts branch from 6b4bb5b to f68b191 Compare April 10, 2025 05:55
@as51340 as51340 removed CI -build=release -test=core Run release build and core tests on push CI -build=release -test=e2e Run release build and e2e tests on push labels Apr 11, 2025
@as51340 as51340 force-pushed the use-only-latest-ts branch from f5bc25d to 26142fb Compare April 11, 2025 07:22
@as51340 as51340 added this pull request to the merge queue Apr 11, 2025
@as51340 as51340 added feature feature Docs needed Docs needed labels Apr 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 11, 2025
@as51340 as51340 mentioned this pull request Apr 11, 2025
11 tasks
@as51340 as51340 enabled auto-merge April 11, 2025 07:51
@as51340 as51340 added this pull request to the merge queue Apr 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 11, 2025
@as51340 as51340 added this pull request to the merge queue Apr 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 11, 2025
@as51340 as51340 added this pull request to the merge queue Apr 11, 2025
Merged via the queue into master with commit e2b90d5 Apr 11, 2025
17 checks passed
@as51340 as51340 deleted the use-only-latest-ts branch April 11, 2025 11:00
@gitbuda gitbuda mentioned this pull request Apr 12, 2025
53 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants