Skip to content

[KQP] Shuffle elimination flag bug fix #16367

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Mar 31, 2025
Merged

[KQP] Shuffle elimination flag bug fix #16367

merged 5 commits into from
Mar 31, 2025

Conversation

@pashandor789
Copy link
Contributor

@pashandor789 pashandor789 commented Mar 27, 2025

Changelog category

  • Not for changelog (changelog entry is not required)

@pashandor789 pashandor789 requested a review from a team as a code owner March 27, 2025 20:03
@github-actions
Copy link

github-actions bot commented Mar 27, 2025

🟢 2025-03-27 20:04:11 UTC The validation of the Pull Request description is successful.

@github-actions
Copy link

github-actions bot commented Mar 27, 2025 •
edited
Loading

⚪ 2025-03-27 20:06:24 UTC Pre-commit check linux-x86_64-relwithdebinfo for f4b0112 has started.
⚪ 2025-03-27 20:06:39 UTC Artifacts will be uploaded here
⚪ 2025-03-27 20:09:28 UTC ya make is running...
🟡 2025-03-27 21:12:25 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?
19407 18019 0 32 1244 112

⚪ 2025-03-27 21:14:09 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-27 21:46:13 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?
220 (only retried tests) 80 0 31 0 109

⚪ 2025-03-27 21:46:21 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-03-27 22:12:12 UTC Some tests failed, follow the links below.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
209 (only retried tests) 73 0 30 0 106

🟢 2025-03-27 22:12:19 UTC Build successful.
🟢 2025-03-27 22:12:43 UTC ydbd size 2.2 GiB changed* by +304 Bytes, which is < 100.0 KiB vs main: OK

ydbd size dash main: 4a35432 merge: f4b0112 diff diff %
ydbd size 2 344 395 848 Bytes 2 344 396 152 Bytes +304 Bytes +0.000%
ydbd stripped size 490 255 168 Bytes 490 255 232 Bytes +64 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 Mar 27, 2025 •
edited
Loading

⚪ 2025-03-27 20:06:54 UTC Pre-commit check linux-x86_64-release-asan for f4b0112 has started.
⚪ 2025-03-27 20:07:10 UTC Artifacts will be uploaded here
⚪ 2025-03-27 20:09:55 UTC ya make is running...
🟡 2025-03-27 21:29:22 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?
12079 11921 0 114 11 33

⚪ 2025-03-27 21:30:26 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-27 21:55:41 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?
229 (only retried tests) 179 0 13 6 31

⚪ 2025-03-27 21:55:52 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-03-27 22:22:15 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?
87 (only retried tests) 39 0 12 5 31

🟢 2025-03-27 22:22:21 UTC Build successful.
🟢 2025-03-27 22:22:51 UTC ydbd size 3.8 GiB changed* by +208 Bytes, which is < 100.0 KiB vs main: OK

ydbd size dash main: 4a35432 merge: f4b0112 diff diff %
ydbd size 4 113 860 072 Bytes 4 113 860 280 Bytes +208 Bytes +0.000%
ydbd stripped size 1 421 614 248 Bytes 1 421 614 248 Bytes 0 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

@pashandor789 pashandor789 requested a review from a team as a code owner March 28, 2025 09:05
@github-actions
Copy link

github-actions bot commented Mar 28, 2025 •
edited
Loading

⚪ 2025-03-28 09:08:58 UTC Pre-commit check linux-x86_64-release-asan for 811eec6 has started.
⚪ 2025-03-28 09:09:14 UTC Artifacts will be uploaded here
⚪ 2025-03-28 09:12:11 UTC ya make is running...
🟡 2025-03-28 10:33:51 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?
12363 12223 0 89 17 34

⚪ 2025-03-28 10:35:01 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-28 10:49: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?
195 (only retried tests) 151 0 8 3 33

⚪ 2025-03-28 10:49:15 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-03-28 11:01:36 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?
71 (only retried tests) 35 0 5 3 28

🟢 2025-03-28 11:01:45 UTC Build successful.
🟢 2025-03-28 11:02:19 UTC ydbd size 3.8 GiB changed* by +21.3 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 201a31f merge: 811eec6 diff diff %
ydbd size 4 118 515 528 Bytes 4 118 537 352 Bytes +21.3 KiB +0.001%
ydbd stripped size 1 423 530 056 Bytes 1 423 538 824 Bytes +8.6 KiB +0.001%

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

@pashandor789 pashandor789 force-pushed the CBOCBOCBOCBOCBOCBOLLLL branch from 2b26f94 to b7130eb Compare March 28, 2025 11:28
@github-actions
Copy link

github-actions bot commented Mar 28, 2025 •
edited
Loading

⚪ 2025-03-28 11:35:33 UTC Pre-commit check linux-x86_64-relwithdebinfo for 5510f02 has started.
⚪ 2025-03-28 11:35:36 UTC Artifacts will be uploaded here
⚪ 2025-03-28 11:38:19 UTC ya make is running...
🟡 2025-03-28 12:41:34 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?
26808 24152 0 8 2542 106

⚪ 2025-03-28 12:44:41 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-28 12:57:12 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?
162 (only retried tests) 57 0 6 0 99

⚪ 2025-03-28 12:57:25 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-03-28 13:11:08 UTC Some tests failed, follow the links below.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
146 (only retried tests) 42 0 6 0 98

🟢 2025-03-28 13:11:17 UTC Build successful.
🟢 2025-03-28 13:11:40 UTC ydbd size 2.2 GiB changed* by +16.7 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9fb5600 merge: 5510f02 diff diff %
ydbd size 2 347 157 296 Bytes 2 347 174 416 Bytes +16.7 KiB +0.001%
ydbd stripped size 490 811 520 Bytes 490 818 432 Bytes +6.8 KiB +0.001%

*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 Mar 28, 2025 •
edited
Loading

⚪ 2025-03-28 11:37:14 UTC Pre-commit check linux-x86_64-release-asan for 5510f02 has started.
⚪ 2025-03-28 11:37:18 UTC Artifacts will be uploaded here
⚪ 2025-03-28 11:40:13 UTC ya make is running...
🟡 2025-03-28 12:55:12 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?
12399 12269 0 80 16 34

⚪ 2025-03-28 12:56:26 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-28 13:11:10 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?
180 (only retried tests) 108 0 38 4 30

⚪ 2025-03-28 13:11:19 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-03-28 13:24:10 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?
99 (only retried tests) 32 0 34 5 28

🟢 2025-03-28 13:24:17 UTC Build successful.
🟢 2025-03-28 13:24:47 UTC ydbd size 3.8 GiB changed* by +17.3 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9fb5600 merge: 5510f02 diff diff %
ydbd size 4 119 149 616 Bytes 4 119 167 344 Bytes +17.3 KiB +0.000%
ydbd stripped size 1 423 819 368 Bytes 1 423 824 040 Bytes +4.6 KiB +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 Mar 28, 2025 •
edited
Loading

⚪ 2025-03-28 14:11:48 UTC Pre-commit check linux-x86_64-relwithdebinfo for 4836b75 has started.
⚪ 2025-03-28 14:12:11 UTC Artifacts will be uploaded here
⚪ 2025-03-28 14:15:27 UTC ya make is running...
🟡 2025-03-28 15:18:03 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?
26811 24147 0 7 2542 115

⚪ 2025-03-28 15:20:37 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-28 15:34:18 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?
164 (only retried tests) 61 0 4 0 99

⚪ 2025-03-28 15:34:26 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-03-28 15:49:04 UTC Some tests failed, follow the links below.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
143 (only retried tests) 42 0 4 0 97

🟢 2025-03-28 15:49:16 UTC Build successful.
🟢 2025-03-28 15:49:38 UTC ydbd size 2.2 GiB changed* by +20.0 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 3eed4cc merge: 4836b75 diff diff %
ydbd size 2 347 444 256 Bytes 2 347 464 696 Bytes +20.0 KiB +0.001%
ydbd stripped size 490 843 264 Bytes 490 847 616 Bytes +4.2 KiB +0.001%

*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 Mar 28, 2025 •
edited
Loading

⚪ 2025-03-28 14:16:16 UTC Pre-commit check linux-x86_64-release-asan for 4836b75 has started.
⚪ 2025-03-28 14:16:31 UTC Artifacts will be uploaded here
⚪ 2025-03-28 14:19:20 UTC ya make is running...
🟡 2025-03-28 15:35:00 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?
12399 12183 0 167 16 33

⚪ 2025-03-28 15:36:12 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-28 15:49:21 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?
268 (only retried tests) 198 0 32 8 30

⚪ 2025-03-28 15:49:37 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-03-28 16:08:39 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?
99 (only retried tests) 31 0 32 7 29

🟢 2025-03-28 16:08:53 UTC Build successful.
🟢 2025-03-28 16:09:27 UTC ydbd size 3.8 GiB changed* by +28.7 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 3eed4cc merge: 4836b75 diff diff %
ydbd size 4 119 585 304 Bytes 4 119 614 688 Bytes +28.7 KiB +0.001%
ydbd stripped size 1 423 878 696 Bytes 1 423 887 336 Bytes +8.4 KiB +0.001%

*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 Mar 29, 2025 •
edited
Loading

⚪ 2025-03-29 23:35:31 UTC Pre-commit check linux-x86_64-relwithdebinfo for 1b629c1 has started.
⚪ 2025-03-29 23:35:45 UTC Artifacts will be uploaded here
⚪ 2025-03-29 23:38:30 UTC ya make is running...
🔴 2025-03-29 23:41:27 UTC Build failed, see the logs. Also see fail summary

@github-actions
Copy link

github-actions bot commented Mar 29, 2025 •
edited
Loading

⚪ 2025-03-29 23:46:44 UTC Pre-commit check linux-x86_64-release-asan for 1b629c1 has started.
⚪ 2025-03-29 23:46:59 UTC Artifacts will be uploaded here
⚪ 2025-03-29 23:49:50 UTC ya make is running...
⚫ 2025-03-29 23:51:53 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Mar 29, 2025 •
edited
Loading

⚪ 2025-03-29 23:55:17 UTC Pre-commit check linux-x86_64-relwithdebinfo for ae55207 has started.
⚪ 2025-03-29 23:55:31 UTC Artifacts will be uploaded here
⚪ 2025-03-29 23:58:20 UTC ya make is running...
🟡 2025-03-30 01:01:28 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?
26931 24260 0 3 2556 112

⚪ 2025-03-30 01:03:44 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-03-30 01:14:09 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
165 (only retried tests) 66 0 0 0 99

🟢 2025-03-30 01:14:16 UTC Build successful.
🟢 2025-03-30 01:14:35 UTC ydbd size 2.2 GiB changed* by +21.4 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: a2c3448 merge: ae55207 diff diff %
ydbd size 2 348 813 160 Bytes 2 348 835 088 Bytes +21.4 KiB +0.001%
ydbd stripped size 491 057 568 Bytes 491 062 688 Bytes +5.0 KiB +0.001%

*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 Mar 29, 2025 •
edited
Loading

⚪ 2025-03-29 23:55:24 UTC Pre-commit check linux-x86_64-release-asan for ae55207 has started.
⚪ 2025-03-29 23:55:39 UTC Artifacts will be uploaded here
⚪ 2025-03-29 23:58:26 UTC ya make is running...
🟡 2025-03-30 01:11:57 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?
12511 12387 0 59 28 37

⚪ 2025-03-30 01:13:01 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-03-30 01:25:22 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?
164 (only retried tests) 99 0 32 2 31

⚪ 2025-03-30 01:25:33 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-03-30 01:36:56 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?
95 (only retried tests) 32 0 32 1 30

🟢 2025-03-30 01:37:05 UTC Build successful.
🟢 2025-03-30 01:37:34 UTC ydbd size 3.8 GiB changed* by +31.3 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: a2c3448 merge: ae55207 diff diff %
ydbd size 4 121 435 704 Bytes 4 121 467 736 Bytes +31.3 KiB +0.001%
ydbd stripped size 1 424 307 656 Bytes 1 424 317 640 Bytes +9.8 KiB +0.001%

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

@pashandor789 pashandor789 enabled auto-merge (squash) March 30, 2025 11:01
@pashandor789 pashandor789 merged commit c0bd909 into ydb-platform:main Mar 31, 2025
12 checks passed
pashandor789 added a commit to pashandor789/ydb that referenced this pull request May 5, 2025
@pashandor789
[KQP] Shuffle elimination flag bug fix (ydb-platform#16367)
8d116ce
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pavelvelikhov pavelvelikhov pavelvelikhov approved these changes

Assignees

No one assigned

Labels

not-for-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pashandor789 @pavelvelikhov