Skip to content

fix sl on empty inputs #27588

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 1 commit into from
Oct 27, 2025
Merged

fix sl on empty inputs #27588

merged 1 commit into from
Oct 27, 2025

Conversation

@gridnevvvit
Copy link
Member

@gridnevvvit gridnevvvit commented Oct 24, 2025

Changelog entry

Changelog category

  • Not for changelog (changelog entry is not required)

Description for reviewers

...

@gridnevvvit gridnevvvit requested a review from a team as a code owner October 24, 2025 18:28
@github-actions
Copy link

github-actions bot commented Oct 24, 2025 •
edited
Loading

⚪ 2025-10-24 18:29:55 UTC Pre-commit check linux-x86_64-release-asan for 53bce02 has started.
⚪ 2025-10-24 18:30:10 UTC Artifacts will be uploaded here
⚪ 2025-10-24 18:33:07 UTC ya make is running...

@github-actions
Copy link

github-actions bot commented Oct 24, 2025

🟢 2025-10-24 18:31:01 UTC The validation of the Pull Request description is successful.

@github-actions
Copy link

github-actions bot commented Oct 24, 2025 •
edited
Loading

⚪ 2025-10-24 18:39:47 UTC Pre-commit check linux-x86_64-relwithdebinfo for 53bce02 has started.
⚪ 2025-10-24 18:40:04 UTC Artifacts will be uploaded here
⚪ 2025-10-24 18:43:45 UTC ya make is running...
🟡 2025-10-24 20:30:16 UTC Some tests failed, follow the links below. Going to retry failed tests...

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39051 36237 0 9 2780 25

⚪ 2025-10-24 20:30:28 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-10-24 20:55:44 UTC Some tests failed, follow the links below. Going to retry failed tests...

Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
901 (only retried tests) 873 0 1 3 24

⚪ 2025-10-24 20:55:47 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-10-24 21:12:54 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
279 (only retried tests) 258 0 0 0 21

🟢 2025-10-24 21:12:57 UTC Build successful.
🟢 2025-10-24 21:13:15 UTC ydbd size 2.3 GiB changed* by +704 Bytes, which is < 100.0 KiB vs main: OK

ydbd size dash main: cf7b87d merge: 53bce02 diff diff %
ydbd size 2 437 327 768 Bytes 2 437 328 472 Bytes +704 Bytes +0.000%
ydbd stripped size 518 128 584 Bytes 518 128 840 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

@gridnevvvit
Copy link
Member Author

gridnevvvit commented Oct 24, 2025 •
edited
Loading

fixes #27589

@gridnevvvit gridnevvvit enabled auto-merge (squash) October 24, 2025 21:56
@gridnevvvit gridnevvvit disabled auto-merge October 26, 2025 14:43
@gridnevvvit gridnevvvit added the rebase-and-check Rebase PR with the current base branch and check label Oct 26, 2025
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label Oct 26, 2025
@github-actions
Copy link

github-actions bot commented Oct 26, 2025 •
edited
Loading

⚪ 2025-10-26 14:46:13 UTC Pre-commit check linux-x86_64-relwithdebinfo for d208a03 has started.
⚪ 2025-10-26 14:46:27 UTC Artifacts will be uploaded here
⚪ 2025-10-26 14:50:06 UTC ya make is running...
🟡 2025-10-26 16:33:16 UTC Some tests failed, follow the links below. Going to retry failed tests...

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39053 36248 0 10 2763 32

⚪ 2025-10-26 16:33:30 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-10-26 16:54:18 UTC Some tests failed, follow the links below. Going to retry failed tests...

Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
601 (only retried tests) 573 0 2 1 25

⚪ 2025-10-26 16:54:21 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-10-26 17:14:35 UTC Some tests failed, follow the links below.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
478 (only retried tests) 456 0 1 0 21

🟢 2025-10-26 17:14:38 UTC Build successful.
🟢 2025-10-26 17:14:58 UTC ydbd size 2.3 GiB changed* by +704 Bytes, which is < 100.0 KiB vs main: OK

ydbd size dash main: fd2cb7b merge: d208a03 diff diff %
ydbd size 2 437 356 520 Bytes 2 437 357 224 Bytes +704 Bytes +0.000%
ydbd stripped size 518 133 704 Bytes 518 133 960 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 Oct 26, 2025 •
edited
Loading

⚪ 2025-10-26 14:49:22 UTC Pre-commit check linux-x86_64-release-asan for d208a03 has started.
⚪ 2025-10-26 14:49:26 UTC Artifacts will be uploaded here
⚪ 2025-10-26 14:52:23 UTC ya make is running...
🟡 2025-10-26 16:57:53 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15516 14900 0 306 289 21

🟢 2025-10-26 16:57:58 UTC Build successful.
🟢 2025-10-26 16:58:20 UTC ydbd size 3.8 GiB changed* by +2.1 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: fd2cb7b merge: d208a03 diff diff %
ydbd size 4 055 777 320 Bytes 4 055 779 456 Bytes +2.1 KiB +0.000%
ydbd stripped size 1 506 056 128 Bytes 1 506 056 896 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

@gridnevvvit gridnevvvit merged commit dbcbf39 into ydb-platform:main Oct 27, 2025
13 of 17 checks passed
@gridnevvvit gridnevvvit deleted the add--ut-filter-optional-side branch October 27, 2025 09:20
@denis0x0D denis0x0D mentioned this pull request Oct 27, 2025
Open
@serbel324 serbel324 mentioned this pull request Oct 30, 2025
Merged
gridnevvvit added a commit to gridnevvvit/ydb that referenced this pull request Nov 11, 2025
@gridnevvvit
fix sl on empty inputs (ydb-platform#27588)
ac73af5
gridnevvvit added a commit to gridnevvvit/ydb that referenced this pull request Nov 11, 2025
@gridnevvvit
fix sl on empty inputs (ydb-platform#27588)
9576a8e
qyryq pushed a commit to qyryq/ydb that referenced this pull request Nov 24, 2025
@gridnevvvit @qyryq
fix sl on empty inputs (ydb-platform#27588)
2450538
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@denis0x0D denis0x0D denis0x0D 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

@gridnevvvit @denis0x0D