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

Yql-18391 gracejoin refactoring for spilling #4559

Conversation

lll-phill-lll
Copy link
Member

Changelog entry

Small refactoring that makes its easier to add spilling to grace join

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

@lll-phill-lll lll-phill-lll requested a review from a team as a code owner May 15, 2024 14:02
Copy link

github-actions bot commented May 15, 2024

2024-05-15 14:03:27 UTC Pre-commit check for 594b374 has started.
2024-05-15 14:03:29 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-15 14:07:20 UTC Build successful.

Copy link

github-actions bot commented May 15, 2024

2024-05-15 14:04:32 UTC Pre-commit check for 594b374 has started.
2024-05-15 14:04:33 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-15 14:07:43 UTC Build successful.
2024-05-15 14:09:21 UTC Tests are running...
🔴 2024-05-15 16:05:32 UTC Test run completed, no test results found for commit 35a1f44. Please check build logs.
2024-05-15 16:05:35 UTC Check cancelled

Copy link

github-actions bot commented May 15, 2024

2024-05-15 14:05:26 UTC Pre-commit check for 594b374 has started.
2024-05-15 14:05:28 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-15 14:08:12 UTC Build successful.
2024-05-15 14:10:06 UTC Tests are running...
🔴 2024-05-15 15:52:45 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11698 11583 0 37 69 9

Copy link

github-actions bot commented May 15, 2024

2024-05-15 16:09:04 UTC Pre-commit check for 36d3763 has started.
2024-05-15 16:09:06 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-15 16:11:54 UTC Build successful.
2024-05-15 16:13:52 UTC Tests are running...
🔴 2024-05-15 16:47:43 UTC Test run completed, no test results found for commit a77f3f4. Please check build logs.
2024-05-15 16:47:46 UTC Check cancelled

Copy link

github-actions bot commented May 15, 2024

2024-05-15 16:09:09 UTC Pre-commit check for 36d3763 has started.
2024-05-15 16:09:11 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-15 16:12:35 UTC Build successful.

Copy link

github-actions bot commented May 15, 2024

2024-05-15 16:09:18 UTC Pre-commit check for 36d3763 has started.
2024-05-15 16:09:20 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-15 16:12:34 UTC Build successful.
2024-05-15 16:14:27 UTC Tests are running...
🔴 2024-05-15 16:47:55 UTC Test run completed, no test results found for commit a77f3f4. Please check build logs.
2024-05-15 16:47:59 UTC Check cancelled

Copy link

github-actions bot commented May 15, 2024

2024-05-15 16:49:18 UTC Pre-commit check for 4bfdde1 has started.
2024-05-15 16:49:19 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-15 16:52:58 UTC Build successful.

Copy link

github-actions bot commented May 15, 2024

2024-05-15 16:51:28 UTC Pre-commit check for 4bfdde1 has started.
2024-05-15 16:51:31 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-15 16:54:22 UTC Build successful.
2024-05-15 16:56:20 UTC Tests are running...
🔴 2024-05-15 18:42:47 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11699 11601 0 33 56 9

Copy link

github-actions bot commented May 15, 2024

2024-05-15 16:52:34 UTC Pre-commit check for 4bfdde1 has started.
2024-05-15 16:52:37 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-15 16:55:55 UTC Build successful.
2024-05-15 16:57:52 UTC Tests are running...
🔴 2024-05-15 18:57:02 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
70526 57661 0 203 12654 8

@lll-phill-lll lll-phill-lll force-pushed the YQL-18391-gracejoin-refactoring-for-spilling branch from badd59a to 994685b Compare May 16, 2024 08:27
Copy link

github-actions bot commented May 16, 2024

2024-05-16 08:30:53 UTC Pre-commit check for 1dbdfcb has started.
2024-05-16 08:30:55 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-16 08:34:17 UTC Build successful.

Copy link

github-actions bot commented May 16, 2024

2024-05-16 08:30:54 UTC Pre-commit check for 1dbdfcb has started.
2024-05-16 08:30:57 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-16 08:33:50 UTC Build successful.
2024-05-16 08:35:51 UTC Tests are running...
🔴 2024-05-16 10:23:08 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11699 11617 0 29 44 9

Copy link

github-actions bot commented May 16, 2024

2024-05-16 08:31:02 UTC Pre-commit check for 1dbdfcb has started.
2024-05-16 08:31:04 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-16 08:34:22 UTC Build successful.
2024-05-16 08:36:17 UTC Tests are running...
🔴 2024-05-16 10:33:37 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
70530 57654 0 212 12654 10

Copy link

github-actions bot commented May 16, 2024

2024-05-16 13:32:54 UTC Pre-commit check for 309b8b7 has started.
2024-05-16 13:32:56 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-16 13:35:51 UTC Build successful.
2024-05-16 13:37:45 UTC Tests are running...
🔴 2024-05-16 15:21:39 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11710 11463 0 47 193 7


if (NumberOfStringColumns || NumberOfIColumns ) {
stringsOffsets.push_back(offset); // Adding offset to tuple in keyIntVals vector
stringsOffsets.push_back(stringVals.size()); // Adding offset to string values
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these 2 sizes will be less than full vector size because of spilling. Vectors partly will be spilled.

Copy link

github-actions bot commented May 16, 2024

2024-05-16 13:52:48 UTC Pre-commit check for 309b8b7 has started.
2024-05-16 13:52:50 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-16 13:55:56 UTC Build successful.

Copy link

github-actions bot commented May 16, 2024

2024-05-16 13:55:50 UTC Pre-commit check for 309b8b7 has started.
2024-05-16 13:55:52 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-16 13:58:45 UTC Build successful.
2024-05-16 14:00:24 UTC Tests are running...
🔴 2024-05-16 15:47:30 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
70532 57646 0 208 12671 7

@lll-phill-lll lll-phill-lll requested a review from Darych May 16, 2024 13:59
Copy link

github-actions bot commented May 17, 2024

2024-05-17 06:22:18 UTC Pre-commit check for 9bd5332 has started.
2024-05-17 06:22:21 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-17 06:58:17 UTC Build successful.

Copy link

github-actions bot commented May 17, 2024

2024-05-17 10:25:49 UTC Pre-commit check for 9bd5332 has started.
2024-05-17 10:25:53 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-17 10:27:07 UTC Build successful.
2024-05-17 10:29:07 UTC Tests are running...
🔴 2024-05-17 11:25:46 UTC Test run completed, no test results found for commit 64cc097. Please check build logs.
2024-05-17 11:25:49 UTC Check cancelled

Copy link

github-actions bot commented May 17, 2024

2024-05-17 10:25:52 UTC Pre-commit check for 9bd5332 has started.
2024-05-17 10:25:55 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-17 10:27:08 UTC Build successful.
2024-05-17 10:29:06 UTC Tests are running...
🔴 2024-05-17 11:25:45 UTC Test run completed, no test results found for commit 64cc097. Please check build logs.
2024-05-17 11:25:49 UTC Check cancelled

Darych
Darych previously approved these changes May 17, 2024
@lll-phill-lll lll-phill-lll force-pushed the YQL-18391-gracejoin-refactoring-for-spilling branch from 64cc097 to 2a7a3a8 Compare May 17, 2024 11:25
Copy link

github-actions bot commented May 17, 2024

2024-05-17 11:27:09 UTC Pre-commit check for 33fa0a3 has started.
2024-05-17 11:27:11 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-17 11:39:17 UTC Build successful.
2024-05-17 11:41:03 UTC Tests are running...
🔴 2024-05-17 13:42:54 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11757 11413 0 65 267 12

Copy link

✅ Documentation build

Revision built successfully
Revision preview link

Copy link

github-actions bot commented May 17, 2024

2024-05-17 11:35:05 UTC Pre-commit check for 33fa0a3 has started.
2024-05-17 11:35:06 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-17 11:48:05 UTC Build successful.

Copy link

github-actions bot commented May 17, 2024

2024-05-17 11:38:57 UTC Pre-commit check for 33fa0a3 has started.
2024-05-17 11:38:59 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-17 11:42:00 UTC Build successful.
2024-05-17 11:43:44 UTC Tests are running...
🔴 2024-05-17 13:37:10 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
70573 57871 0 12 12681 9

@lll-phill-lll lll-phill-lll force-pushed the YQL-18391-gracejoin-refactoring-for-spilling branch from 78199b2 to 74f7d9b Compare May 17, 2024 17:17
Copy link

github-actions bot commented May 17, 2024

2024-05-17 17:20:37 UTC Pre-commit check for ca9ac8b has started.
2024-05-17 17:20:40 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-17 17:21:49 UTC Build successful.
2024-05-17 17:23:36 UTC Tests are running...
🔴 2024-05-17 19:20:40 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11767 11506 0 45 192 24

Copy link

github-actions bot commented May 17, 2024

2024-05-17 17:20:48 UTC Pre-commit check for ca9ac8b has started.
2024-05-17 17:20:50 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-17 17:22:00 UTC Build successful.

Copy link

github-actions bot commented May 17, 2024

2024-05-17 17:22:39 UTC Pre-commit check for ca9ac8b has started.
2024-05-17 17:22:41 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-17 17:23:40 UTC Build successful.
2024-05-17 17:25:15 UTC Tests are running...
🔴 2024-05-17 19:14:34 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
70603 57922 0 2 12670 9

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.

None yet

2 participants