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

Remove MvccTestOutOfOrderRestartLocksSingleWithoutBarrier #1905

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

snaury
Copy link
Member

@snaury snaury commented Feb 13, 2024

Changelog entry

Remove MvccTestOutOfOrderRestartLocksSingleWithoutBarrier test.

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

It doesn't test anything useful with volatile transactions enabled, and broken with volatile transactions disabled due to past dependence on disabling a removed feature flag.

Copy link

github-actions bot commented Feb 13, 2024

2024-02-13 20:20:10 UTC Pre-commit check for f3c5c33 has started.
2024-02-13 20:20:12 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-13 20:23:03 UTC Build successful.
2024-02-13 20:23:15 UTC Tests are running...
🔴 2024-02-13 20:28:28 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14761 14550 0 17 148 46

Copy link

github-actions bot commented Feb 13, 2024

2024-02-13 20:20:14 UTC Pre-commit check for f3c5c33 has started.
2024-02-13 20:20:17 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-13 20:22:56 UTC Build successful.
2024-02-13 20:23:12 UTC Tests are running...
🔴 2024-02-13 20:35:12 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67489 56458 0 16 10971 44

@snaury snaury requested a review from azevaykin February 14, 2024 08:57
@snaury snaury marked this pull request as ready for review February 14, 2024 08:58
@snaury snaury merged commit 787a642 into ydb-platform:main Feb 14, 2024
@snaury snaury deleted the datashard-remove-broken-test branch February 14, 2024 09:03
@vitstn vitstn mentioned this pull request Feb 16, 2024
@mvgorbunov mvgorbunov mentioned this pull request Feb 22, 2024
@shnikd shnikd mentioned this pull request Feb 27, 2024
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.

2 participants