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.

Sign up for GitHub

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

Jump to bottom

Fix flaky test testFlushThrowsFlushFailedExceptionOnCorruption. #10671

Merged
merged 1 commit into from
Oct 17, 2023
Merged

Fix flaky test testFlushThrowsFlushFailedExceptionOnCorruption. #10671

merged 1 commit into from
Oct 17, 2023

Commits on Oct 17, 2023

  1. Fix flaky test testFlushThrowsFlushFailedExceptionOnCorruption. 

    This test is flaky in two ways, one it may hit a case where a merge occurs and the hardcoded segment does not exist to delete.
Two with certain mock filesystems a corruption exception won't be thrown here, only a NoSuchFile - This doesn't change our
handling in the engine, but the test should conditionally handle if the store is marked corrupted on close.

Signed-off-by: Marc Handalian <[email protected]>
    @mch2
    mch2 committed Oct 17, 2023
    Configuration menu
    Copy the full SHA
    8036897 View commit details
    Browse the repository at this point in the history