Skip to content

Conversation

@yumkam
Copy link
Collaborator

@yumkam yumkam commented Dec 9, 2024

Changelog entry

...

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

`["foo"] + ["bar"] if baz else []` is same as `(["foo"] + ["bar"]) if baz else []`

Hence, no-spilling was run without any pragma override
1) avoid comparing apples with oranges: use same pragma set for spilling
   and no spilling, except for spilling pragmas
2) enable spilling in channels for spilling variant
@yumkam yumkam marked this pull request as ready for review December 9, 2024 15:20
@github-actions
Copy link

github-actions bot commented Dec 9, 2024

2024-12-09 15:21:30 UTC Pre-commit check linux-x86_64-relwithdebinfo for 87f1c24 has started.
2024-12-09 15:21:41 UTC Artifacts will be uploaded here
2024-12-09 15:24:14 UTC ya make is running...
🟢 2024-12-09 15:26:27 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
20 20 0 0 0 0

🟢 2024-12-09 15:26:33 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Dec 9, 2024

2024-12-09 15:21:54 UTC Pre-commit check linux-x86_64-release-asan for 87f1c24 has started.
2024-12-09 15:21:58 UTC Artifacts will be uploaded here
2024-12-09 15:24:31 UTC ya make is running...
🟢 2024-12-09 15:27:03 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10 10 0 0 0 0

🟢 2024-12-09 15:27:09 UTC Build successful.

@yumkam yumkam merged commit 7d24286 into ydb-platform:main Dec 9, 2024
11 checks passed
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