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 "feat(backfill): Persist backfill operator state" #9997

Closed
wants to merge 1 commit into from

Conversation

kwannoel
Copy link
Contributor

@kwannoel kwannoel commented May 25, 2023

Reverts #9752, perhaps it causes recovery test to fail, see if recovery still fails if this is reverted.

Also test performance difference if this is reverted.

Testing it on main-cron: https://buildkite.com/risingwavelabs/main-cron/builds/517

@github-actions github-actions bot added Invalid PR Title user-facing-changes Contains changes that are visible to users labels May 25, 2023
@kwannoel kwannoel marked this pull request as draft May 25, 2023 03:56
@kwannoel
Copy link
Contributor Author

Root cause is explained and fixed in #9999.

@kwannoel kwannoel closed this May 25, 2023
@kwannoel kwannoel deleted the revert-9752-kwannoel/persist-backfill branch May 25, 2023 06:11
@CharlieSYH CharlieSYH added the 📖✗ No user documentation is needed. label May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Invalid PR Title user-facing-changes Contains changes that are visible to users 📖✗ No user documentation is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants