Skip to content
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

YQ RD added brackets during pushdown #11742

Merged

Conversation

GrigoriyPA
Copy link
Collaborator

@GrigoriyPA GrigoriyPA commented Nov 19, 2024

Changelog entry

Added brackets during pushdown

Changelog category

  • Bugfix

Additional information

@GrigoriyPA GrigoriyPA requested a review from a team as a code owner November 19, 2024 12:09
@GrigoriyPA GrigoriyPA requested a review from dorooleg November 19, 2024 12:09
@GrigoriyPA GrigoriyPA changed the title YQ RD added barckets during pushdown YQ RD added brackets during pushdown Nov 19, 2024
Copy link

github-actions bot commented Nov 19, 2024

2024-11-19 12:10:28 UTC Pre-commit check linux-x86_64-release-asan for 7968bf7 has started.
2024-11-19 12:10:39 UTC Artifacts will be uploaded here
2024-11-19 12:13:40 UTC ya make is running...
2024-11-19 12:35:15 UTC Check cancelled

Copy link

github-actions bot commented Nov 19, 2024

2024-11-19 12:11:30 UTC Pre-commit check linux-x86_64-relwithdebinfo for 7968bf7 has started.
2024-11-19 12:11:34 UTC Artifacts will be uploaded here
2024-11-19 12:14:37 UTC ya make is running...
2024-11-19 12:35:13 UTC Check cancelled

dorooleg
dorooleg previously approved these changes Nov 19, 2024
@GrigoriyPA GrigoriyPA enabled auto-merge (squash) November 19, 2024 12:23
Copy link

github-actions bot commented Nov 19, 2024

2024-11-19 12:37:31 UTC Pre-commit check linux-x86_64-release-asan for b44da87 has started.
2024-11-19 12:37:35 UTC Artifacts will be uploaded here
2024-11-19 12:40:33 UTC ya make is running...
🟡 2024-11-19 13:40:11 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9023 8951 0 20 7 45

🟢 2024-11-19 13:40:59 UTC Build successful.
🟢 2024-11-19 13:41:26 UTC ydbd size 4.9 GiB changed* by -5.6 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 7365427 merge: b44da87 diff diff %
ydbd size 5 263 270 384 Bytes 5 263 264 640 Bytes -5.6 KiB -0.000%
ydbd stripped size 1 351 576 880 Bytes 1 351 577 136 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

Copy link

github-actions bot commented Nov 19, 2024

2024-11-19 12:37:49 UTC Pre-commit check linux-x86_64-relwithdebinfo for b44da87 has started.
2024-11-19 12:37:53 UTC Artifacts will be uploaded here
2024-11-19 12:40:50 UTC ya make is running...
🟡 2024-11-19 13:34:37 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?
40217 34547 0 1 5559 110

2024-11-19 13:37:59 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-11-19 13:49:14 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
111 (only retried tests) 4 0 0 0 107

🟢 2024-11-19 13:49:20 UTC Build successful.
🟢 2024-11-19 13:49:38 UTC ydbd size 2.5 GiB changed* by -1.1 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 7365427 merge: b44da87 diff diff %
ydbd size 2 685 648 784 Bytes 2 685 647 616 Bytes -1.1 KiB -0.000%
ydbd stripped size 481 258 352 Bytes 481 258 544 Bytes +192 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

@GrigoriyPA GrigoriyPA merged commit 26ed08b into ydb-platform:main Nov 19, 2024
10 checks passed
@GrigoriyPA GrigoriyPA deleted the YQ-RD-add-barckets-during-pushdown branch November 19, 2024 13:50
GrigoriyPA added a commit to GrigoriyPA/ydb that referenced this pull request Nov 19, 2024
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