Skip to content

At the start of a partition, the order of messages may change. #15160

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

Merged
merged 3 commits into from
Feb 28, 2025

Conversation

Alek5andr-Kotov
Copy link
Collaborator

@Alek5andr-Kotov Alek5andr-Kotov commented Feb 27, 2025

Changelog entry

The partition actor puts messages in the queue until it enters the StateIdle state. Then he sends himself these messages again. There may be a situation when new messages will be processed before the accumulated ones.

#15070

Changelog category

  • Bugfix

Description for reviewers

Message processing rule:

  • in the StateInit state, messages are queued up in the PendingEvents queue.
  • in the StateIdle state, messages are processed immediately if the queue is empty and queued otherwise.

After switching from StateInit to StateIdle, all messages in the queue are processed.

Copy link

github-actions bot commented Feb 27, 2025

🟢 2025-02-28 08:32:54 UTC The validation of the Pull Request description is successful.

Copy link

github-actions bot commented Feb 27, 2025

2025-02-27 20:56:59 UTC Pre-commit check linux-x86_64-relwithdebinfo for 910df56 has started.
2025-02-27 20:57:14 UTC Artifacts will be uploaded here
2025-02-27 21:00:00 UTC ya make is running...
🟡 2025-02-27 21:57:16 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
27834 25224 0 6 2474 130

2025-02-27 21:59:37 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-02-27 22:10:50 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
209 (only retried tests) 85 0 0 0 124

🟢 2025-02-27 22:10:57 UTC Build successful.
🟡 2025-02-27 22:11:16 UTC ydbd size 2.1 GiB changed* by +182.2 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: e8002d3 merge: 910df56 diff diff %
ydbd size 2 286 798 992 Bytes 2 286 985 528 Bytes +182.2 KiB +0.008%
ydbd stripped size 479 450 688 Bytes 479 463 104 Bytes +12.1 KiB +0.003%

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

Copy link

github-actions bot commented Feb 27, 2025

2025-02-27 20:57:05 UTC Pre-commit check linux-x86_64-release-asan for 910df56 has started.
2025-02-27 20:57:19 UTC Artifacts will be uploaded here
2025-02-27 21:00:12 UTC ya make is running...
🟡 2025-02-27 22:11:23 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13465 13291 0 116 20 38

2025-02-27 22:12:37 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-02-27 22:28:04 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
234 (only retried tests) 198 0 7 1 28

2025-02-27 22:28:13 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-02-27 22:37:53 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67 (only retried tests) 38 0 1 0 28

🟢 2025-02-27 22:38:00 UTC Build successful.
🟡 2025-02-27 22:38:27 UTC ydbd size 3.7 GiB changed* by +218.2 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: e8002d3 merge: 910df56 diff diff %
ydbd size 3 985 630 368 Bytes 3 985 853 856 Bytes +218.2 KiB +0.006%
ydbd stripped size 1 387 837 256 Bytes 1 387 879 848 Bytes +41.6 KiB +0.003%

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

Copy link

github-actions bot commented Feb 28, 2025

2025-02-28 06:11:22 UTC Pre-commit check linux-x86_64-relwithdebinfo for 91fe302 has started.
2025-02-28 06:11:38 UTC Artifacts will be uploaded here
2025-02-28 06:14:26 UTC ya make is running...
🟡 2025-02-28 07:08:00 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
27835 25220 0 7 2476 132

2025-02-28 07:10:19 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-02-28 07:21:45 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
219 (only retried tests) 92 0 1 0 126

2025-02-28 07:21:56 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-02-28 07:32:44 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
190 (only retried tests) 67 0 0 0 123

🟢 2025-02-28 07:32:54 UTC Build successful.
🟡 2025-02-28 07:33:16 UTC ydbd size 2.1 GiB changed* by +196.7 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: ce94a36 merge: 91fe302 diff diff %
ydbd size 2 286 816 688 Bytes 2 287 018 120 Bytes +196.7 KiB +0.009%
ydbd stripped size 479 452 800 Bytes 479 468 096 Bytes +14.9 KiB +0.003%

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

Copy link

github-actions bot commented Feb 28, 2025

2025-02-28 06:11:23 UTC Pre-commit check linux-x86_64-release-asan for 91fe302 has started.
2025-02-28 06:11:39 UTC Artifacts will be uploaded here
2025-02-28 06:14:37 UTC ya make is running...
🟡 2025-02-28 07:24:45 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13466 13240 0 174 17 35

2025-02-28 07:25:54 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-02-28 07:44:50 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
290 (only retried tests) 237 0 23 1 29

2025-02-28 07:45:00 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-02-28 07:59:16 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
90 (only retried tests) 62 0 0 0 28

🟢 2025-02-28 07:59:25 UTC Build successful.
🟡 2025-02-28 07:59:55 UTC ydbd size 3.7 GiB changed* by +246.1 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: ce94a36 merge: 91fe302 diff diff %
ydbd size 3 985 651 168 Bytes 3 985 903 176 Bytes +246.1 KiB +0.006%
ydbd stripped size 1 387 843 176 Bytes 1 387 899 336 Bytes +54.8 KiB +0.004%

*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 enabled auto-merge (squash) February 28, 2025 08:20
@github-actions github-actions bot added bugfix and removed bugfix labels Feb 28, 2025
@Alek5andr-Kotov Alek5andr-Kotov merged commit b0acbda into ydb-platform:main Feb 28, 2025
15 of 16 checks passed
@Alek5andr-Kotov Alek5andr-Kotov mentioned this pull request Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants