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

ccl/changefeedccl: deflake TestChangefeedSchemaChangeAllowBackfill #33272

Conversation

petermattis
Copy link
Collaborator

Any goroutine started via Stopper.RunAsyncTask needs to watch
Stopper.ShouldQuiesce() to determine if it should stop
processing. This wasn't being done properly for the "changefeed-poller"
task causing the test to fail to exit in rare circumstances.

Fixes #33166

Release note: None

@petermattis petermattis requested a review from a team December 19, 2018 15:03
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@petermattis petermattis force-pushed the pmattis/deflake-changefeed-schemachange-allow-backfill branch from 51aa732 to c31e3e0 Compare December 19, 2018 15:16
Any goroutine started via `Stopper.RunAsyncTask` needs to watch
`Stopper.ShouldQuiesce()` to determine if it should stop
processing. This wasn't being done properly for the "changefeed-poller"
task causing the test to fail to exit in rare circumstances.

Fixes cockroachdb#33166

Release note: None
@petermattis petermattis force-pushed the pmattis/deflake-changefeed-schemachange-allow-backfill branch from c31e3e0 to 0b1a649 Compare December 19, 2018 15:18
@petermattis
Copy link
Collaborator Author

Hold off on reviewing this. I'm pretty sure this is the source of the wedged tests, but the fix needs some more love.

@petermattis
Copy link
Collaborator Author

Turns out there was an additional problem: a deadlock in rangefeed processor teardown. I have a fix, just need to stress it for a while.

@petermattis
Copy link
Collaborator Author

Turns out that the stopper change wasn't necessary. Closing this in favor of #33303.

@petermattis petermattis deleted the pmattis/deflake-changefeed-schemachange-allow-backfill branch December 20, 2018 16:15
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.

None yet

2 participants