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

CQ: Fix a crash when CMQ master changes in mixed environment test (backport #9507) #9508

Merged
merged 1 commit into from
Sep 21, 2023

Commits on Sep 21, 2023

  1. CQ: Fix a crash when CMQ master changes in mixed environment test

    When policies are applied to change a CMQ master, and that master
    moves from 3.13 to 3.12, this triggers a CQ version change
    (because it defaults to v2 in 3.13 but v1 in 3.12). which
    leads to a crash because the files on disk have an index entry
    for messages that are in memory only.
    
    The index entry is only created with CMQs, normal CQs prohibit
    this behavior. Since we are going to remove CMQs "soon" the
    solution to fix the crash was to simply ignore the index entry
    when we know the message is already in memory.
    
    (cherry picked from commit 90e871d)
    lhoguin authored and mergify[bot] committed Sep 21, 2023
    Configuration menu
    Copy the full SHA
    6bcbcbd View commit details
    Browse the repository at this point in the history