Skip to content

Remove second read stage during uniq index update #1258

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

Conversation

dcherednik
Copy link
Member

Changelog entry

During uniq index update we had 2 read stages:
1 - read to fill missed input collumns.
2 - read to delete old index records.
This patch allows to use 1 to delete old records.

Changelog category

  • Improvement

Additional information

During uniq index update we had 2 read stages:
1 - read to fill missed input collumns.
2 - read to delete old index records.
This patch allows to use 1 to delete old records.
Copy link

github-actions bot commented Jan 24, 2024

2024-01-24 11:56:05 UTC Pre-commit check for da5d17c has started.
2024-01-24 11:56:07 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-24 11:58:53 UTC Build successful.
2024-01-24 11:59:02 UTC Tests are running...
🔴 2024-01-24 13:12:51 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
60028 50761 0 9 9228 30

Copy link

github-actions bot commented Jan 24, 2024

2024-01-24 11:56:31 UTC Pre-commit check for da5d17c has started.
2024-01-24 11:56:32 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-24 11:59:51 UTC Build successful.
2024-01-24 12:00:01 UTC Tests are running...
🔴 2024-01-24 13:35:58 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15920 15814 0 23 59 24

@dcherednik dcherednik merged commit 03e98cb into ydb-platform:main Jan 24, 2024
@niksaveliev niksaveliev mentioned this pull request Jan 29, 2024
@vitstn vitstn mentioned this pull request Feb 5, 2024
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
@dcherednik dcherednik deleted the kqp_reduse_stage_count_1_rebase branch February 19, 2024 18:03
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