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

Add sleep to proper order of statements in fts_segment_reset test #1099

Open
wants to merge 1 commit into
base: adb-7.2.0
Choose a base branch
from

Conversation

dkovalev1
Copy link

@dkovalev1 dkovalev1 commented Oct 30, 2024

Ensure proper order of statements in the test fts_segment_reset.

The aforementioned isolation test contains statements that are started in
parallel at the same time. Under some circumstances, e.g., when using a Docker
container based on Ubuntu 22.04, these statements are executed in an unexpected
order, causing the test to fail.

This commit adds a simple solution to enforce the order of statements by
introducing a delay between the start of commands.

-- Make sure that fault is processed and the following query started after seg0 has actually gone
-- Of course, sleeping is not the best solution, but wait_until_triggered_fault won't work here
-- because seg0 process with faults hash table has gone.
select pg_sleep(0.5);
Copy link
Member

Choose a reason for hiding this comment

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

Can we use something like?

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