Skip to content

Merge to stable-24-3 missing commits for stream join #9566

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

ulya-sidorina
Copy link
Collaborator

@ulya-sidorina ulya-sidorina commented Sep 20, 2024

Missing commits:

  • fix(kqp): return one result row per key from stream join actor for left semi strategy
  • feat(kqp): support stream lookup join for index table

@ulya-sidorina ulya-sidorina linked an issue Sep 20, 2024 that may be closed by this pull request
Copy link

github-actions bot commented Sep 20, 2024

2024-09-20 14:40:17 UTC Pre-commit check for 25236c5 has started.
2024-09-20 14:43:07 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-20 15:20:26 UTC Build successful.

Copy link

github-actions bot commented Sep 20, 2024

2024-09-20 14:40:47 UTC Pre-commit check for 25236c5 has started.
2024-09-20 14:43:43 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-20 15:26:08 UTC Build successful.
2024-09-20 15:26:30 UTC Tests are running...
🔴 2024-09-20 17:28:16 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11368 11295 0 10 28 35

🟡 2024-09-20 17:28:59 UTC ydbd size 5.5 GiB changed* by +453.1 KiB, which is >= 100.0 KiB vs stable-24-3: Warning

ydbd size dash stable-24-3: 7eb941d merge: 25236c5 diff diff %
ydbd size 5 935 286 320 Bytes 5 935 750 344 Bytes +453.1 KiB +0.008%
ydbd stripped size 1 487 994 688 Bytes 1 488 052 960 Bytes +56.9 KiB +0.004%

*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 Sep 20, 2024

2024-09-20 14:40:54 UTC Pre-commit check for 25236c5 has started.
2024-09-20 14:43:56 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-20 15:46:55 UTC Build successful.
2024-09-20 15:47:09 UTC Tests are running...
🔴 2024-09-20 17:35:42 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39921 34668 0 9 5213 31

🟡 2024-09-20 17:36:24 UTC ydbd size 8.2 GiB changed* by +524.6 KiB, which is >= 100.0 KiB vs stable-24-3: Warning

ydbd size dash stable-24-3: 7eb941d merge: 25236c5 diff diff %
ydbd size 8 763 629 544 Bytes 8 764 166 784 Bytes +524.6 KiB +0.006%
ydbd stripped size 479 089 544 Bytes 479 103 240 Bytes +13.4 KiB +0.003%

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

@ulya-sidorina ulya-sidorina changed the title feat(kqp): support stream lookup join for index table Merge to stable-24-3 missing commits for stream join Sep 20, 2024
@ydb-platform ydb-platform deleted a comment from github-actions bot Sep 20, 2024
@ydb-platform ydb-platform deleted a comment from github-actions bot Sep 20, 2024
@ydb-platform ydb-platform deleted a comment from github-actions bot Sep 20, 2024
Copy link

github-actions bot commented Sep 21, 2024

2024-09-21 16:46:52 UTC Pre-commit check for f3d946a has started.
2024-09-21 16:49:42 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-21 17:27:16 UTC Build successful.
2024-09-21 17:27:34 UTC Tests are running...
🔴 2024-09-21 19:11:35 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39921 34669 0 7 5214 31

🟡 2024-09-21 19:12:18 UTC ydbd size 8.2 GiB changed* by +524.7 KiB, which is >= 100.0 KiB vs stable-24-3: Warning

ydbd size dash stable-24-3: 16c4b26 merge: f3d946a diff diff %
ydbd size 8 763 629 352 Bytes 8 764 166 608 Bytes +524.7 KiB +0.006%
ydbd stripped size 479 089 544 Bytes 479 103 240 Bytes +13.4 KiB +0.003%

*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 Sep 21, 2024

2024-09-21 16:46:56 UTC Pre-commit check for f3d946a has started.
2024-09-21 16:49:53 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-21 17:12:42 UTC Build successful.
2024-09-21 17:13:01 UTC Tests are running...
🔴 2024-09-21 19:14:06 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11368 11299 0 9 25 35

🟡 2024-09-21 19:14:51 UTC ydbd size 5.5 GiB changed* by +453.1 KiB, which is >= 100.0 KiB vs stable-24-3: Warning

ydbd size dash stable-24-3: 7eb941d merge: f3d946a diff diff %
ydbd size 5 935 286 320 Bytes 5 935 750 248 Bytes +453.1 KiB +0.008%
ydbd stripped size 1 487 994 688 Bytes 1 488 052 896 Bytes +56.8 KiB +0.004%

*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 Sep 21, 2024

2024-09-21 16:50:45 UTC Pre-commit check for f3d946a has started.
2024-09-21 16:53:43 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-21 17:00:13 UTC Build successful.

@ulya-sidorina ulya-sidorina marked this pull request as ready for review September 22, 2024 10:08
@ulya-sidorina ulya-sidorina requested a review from a team as a code owner September 22, 2024 10:08
@ulya-sidorina ulya-sidorina merged commit 45f8e08 into ydb-platform:stable-24-3 Sep 24, 2024
8 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support stream join for secondary indexes
3 participants