Skip to content

Simplify TEST_F(DBWALTest, FixSyncWalOnObseletedWalWithNewManifestCausingMissingWAL)#11186

Closed
hx235 wants to merge 1 commit intofacebook:mainfrom
hx235:simplify_ut
Closed

Simplify TEST_F(DBWALTest, FixSyncWalOnObseletedWalWithNewManifestCausingMissingWAL)#11186
hx235 wants to merge 1 commit intofacebook:mainfrom
hx235:simplify_ut

Conversation

@hx235
Copy link
Copy Markdown
Contributor

@hx235 hx235 commented Feb 6, 2023

Context/Summary:
Simplify TEST_F(DBWALTest, FixSyncWalOnObseletedWalWithNewManifestCausingMissingWAL) based on #11016 (review) and delete unused sync points.

Test:

@hx235 hx235 changed the title Simplify Simplify TEST_F(DBWALTest, FixSyncWalOnObseletedWalWithNewManifestCausingMissingWAL) Feb 6, 2023
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@hx235 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Contributor

@ajkr ajkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@hx235 merged this pull request in 9b66331.

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.

3 participants