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

Revert "WT-13442 Do not allow out of order truncate commit timestamp" #11031

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

kommiharibabu
Copy link
Contributor

Reverts #11012

Copy link

Thanks for creating a pull request! The below questions and checklist are intended to help with verifying your change is well tested. Response is optional, but if you choose to respond please edit this comment.

What makes this change safe?

A good answer to this question helps the reviewers understand where they should focus their attention, so please consider these questions:

  • Is the change risky or not? Why?
  • What tests are you adding or changing? Why?
  • What existing tests are you relying on?
  • What, if anything, are you concerned about that you'd like the reviewer to focus on?
    References:
  • Risk level guide
  • Testing frameworks

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have made corresponding changes to the documentation (if applicable).
  • I have added/updated tests that demonstrate my fix is effective or that my feature works correctly.

Copy link
Contributor

@raviprakashgiri29 raviprakashgiri29 left a comment

Choose a reason for hiding this comment

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

LGTM

@kommiharibabu kommiharibabu added this pull request to the merge queue Sep 18, 2024
Merged via the queue into develop with commit d3d44d4 Sep 18, 2024
11 checks passed
@kommiharibabu kommiharibabu deleted the revert-11012-wt-13442-truncate-ooo-timestamp branch September 18, 2024 06:50
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.

2 participants