Skip to content

Conversation

@Alek5andr-Kotov
Copy link
Collaborator

@Alek5andr-Kotov Alek5andr-Kotov commented Apr 10, 2025

Changelog entry

Moved changed from #17116

Issue #17118

The message TEvDeletePartition may arrive earlier than TEvApproveWriteQuota. The batch did not send TEvConsumed and this blocked the queue of write quota requests.

Changelog category

  • Bugfix

Description for reviewers

The batch waits for a response from the quoter before processing the TEvDeletePartition.

@Alek5andr-Kotov Alek5andr-Kotov requested a review from a team as a code owner April 10, 2025 16:30
@github-actions
Copy link

github-actions bot commented Apr 10, 2025

2025-04-10 16:31:18 UTC Pre-commit check for 8930f91 has started.
2025-04-10 16:34:01 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2025-04-10 16:37:18 UTC Build failed. see the build logs.
🔴 2025-04-10 16:37:20 UTC Tests run skipped.

@github-actions
Copy link

github-actions bot commented Apr 10, 2025

2025-04-10 16:31:20 UTC Pre-commit check for 8930f91 has started.
2025-04-10 16:33:58 UTC Build linux-x86_64-release-asan is running...
🔴 2025-04-10 16:36:51 UTC Build failed. see the build logs.
🔴 2025-04-10 16:36:53 UTC Tests run skipped.

@github-actions
Copy link

github-actions bot commented Apr 11, 2025

2025-04-11 05:22:04 UTC Pre-commit check for 303ab2f has started.
2025-04-11 05:24:42 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-04-11 06:04:38 UTC Build successful.
2025-04-11 06:04:52 UTC Tests are running...
🔴 2025-04-11 07:33:06 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14800 13440 0 20 1302 38

🟢 2025-04-11 07:34:14 UTC ydbd size 8.3 GiB changed* by +192 Bytes, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: c09bd6c merge: 303ab2f diff diff %
ydbd size 8 898 235 464 Bytes 8 898 235 656 Bytes +192 Bytes +0.000%
ydbd stripped size 485 016 840 Bytes 485 016 968 Bytes +128 Bytes +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@github-actions
Copy link

github-actions bot commented Apr 11, 2025

2025-04-11 05:22:12 UTC Pre-commit check for 303ab2f has started.
2025-04-11 05:24:47 UTC Build linux-x86_64-release-asan is running...
🟢 2025-04-11 05:48:43 UTC Build successful.
2025-04-11 05:49:05 UTC Tests are running...
🔴 2025-04-11 07:53:34 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10367 10268 0 30 27 42

🟢 2025-04-11 07:54:26 UTC ydbd size 5.6 GiB changed* by +704 Bytes, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: c09bd6c merge: 303ab2f diff diff %
ydbd size 6 027 651 392 Bytes 6 027 652 096 Bytes +704 Bytes +0.000%
ydbd stripped size 1 508 441 344 Bytes 1 508 441 856 Bytes +512 Bytes +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

alexnick88
alexnick88 previously approved these changes Apr 11, 2025
@github-actions
Copy link

github-actions bot commented Apr 11, 2025

2025-04-11 12:26:21 UTC Pre-commit check for 3239b83 has started.
2025-04-11 12:29:01 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-04-11 13:09:54 UTC Build successful.
2025-04-11 13:10:09 UTC Tests are running...
🔴 2025-04-11 14:38:54 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14802 13443 0 17 1302 40

🟢 2025-04-11 14:39:41 UTC ydbd size 8.3 GiB changed* by +1.4 KiB, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: c09bd6c merge: 3239b83 diff diff %
ydbd size 8 898 235 464 Bytes 8 898 236 856 Bytes +1.4 KiB +0.000%
ydbd stripped size 485 016 840 Bytes 485 017 096 Bytes +256 Bytes +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@github-actions
Copy link

github-actions bot commented Apr 11, 2025

2025-04-11 12:27:38 UTC Pre-commit check for 3239b83 has started.
2025-04-11 12:30:13 UTC Build linux-x86_64-release-asan is running...
🟢 2025-04-11 12:54:05 UTC Build successful.
2025-04-11 12:54:24 UTC Tests are running...
🔴 2025-04-11 15:02:43 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10369 10271 0 32 26 40

🟢 2025-04-11 15:03:36 UTC ydbd size 5.6 GiB changed* by +1.2 KiB, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: c09bd6c merge: 3239b83 diff diff %
ydbd size 6 027 651 392 Bytes 6 027 652 592 Bytes +1.2 KiB +0.000%
ydbd stripped size 1 508 441 344 Bytes 1 508 442 112 Bytes +768 Bytes +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@Alek5andr-Kotov Alek5andr-Kotov changed the title The race between events The commit of a transaction in the SDK freezes Apr 11, 2025
@Alek5andr-Kotov Alek5andr-Kotov enabled auto-merge (squash) April 11, 2025 13:11
@Alek5andr-Kotov Alek5andr-Kotov merged commit a77c52e into ydb-platform:stable-24-4 Apr 14, 2025
6 of 10 checks passed
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.

3 participants