Skip to content

Fix YQL-18475#4937

Merged
aakulaga-ydb merged 1 commit intoydb-platform:mainfrom
yumkam:YQL-18475
May 31, 2024
Merged

Fix YQL-18475#4937
aakulaga-ydb merged 1 commit intoydb-platform:mainfrom
yumkam:YQL-18475

Conversation

@yumkam
Copy link
Collaborator

@yumkam yumkam commented May 28, 2024

ProcessSpilledData -> NextJoinedData -> HasMoreTuples -> iterates bucketId (refrerencing JoinTable1->CurrIterBucket) out of un-spilled data -> SIGSEGV in GetTupleData on access to empty/spilled TableBuckets[bucketNum].keyIntVals

Changelog entry

...

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

ProcessSpilledData -> NextJoinedData -> HasMoreTuples -> iterates
bucketId (refrerencing JoinTable1->CurrIterBucket) out of un-spilled
data -> SIGSEGV in GetTupleData on access to empty/spilled
TableBuckets[bucketNum].keyIntVals
@github-actions
Copy link

github-actions bot commented May 28, 2024

2024-05-28 13:34:03 UTC Pre-commit check for 49ecaa7 has started.
2024-05-28 13:36:25 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-28 13:39:21 UTC Build successful.
2024-05-28 13:39:32 UTC Tests are running...
🔴 2024-05-28 15:29:35 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
70921 58209 0 2 12702 8

@github-actions
Copy link

github-actions bot commented May 28, 2024

2024-05-28 13:34:32 UTC Pre-commit check for 49ecaa7 has started.
2024-05-28 13:37:05 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-28 13:40:23 UTC Build successful.

@github-actions
Copy link

github-actions bot commented May 28, 2024

2024-05-28 13:34:54 UTC Pre-commit check for 49ecaa7 has started.
2024-05-28 13:37:24 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-28 13:40:00 UTC Build successful.
2024-05-28 13:40:09 UTC Tests are running...
🔴 2024-05-28 15:39:50 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11799 11683 0 39 60 17

@yumkam yumkam marked this pull request as ready for review May 29, 2024 07:46
@yumkam yumkam requested a review from a team as a code owner May 29, 2024 07:46
@aakulaga-ydb aakulaga-ydb merged commit 8c0f1bc into ydb-platform:main May 31, 2024
@StekPerepolnen StekPerepolnen mentioned this pull request May 31, 2024
@niksaveliev niksaveliev mentioned this pull request Jun 17, 2024
@StekPerepolnen StekPerepolnen mentioned this pull request Jun 19, 2024
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.

3 participants