Skip to content

Fix "too large allocation" message from MSan#44613

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-too-large-allocation
Dec 27, 2022
Merged

Fix "too large allocation" message from MSan#44613
alexey-milovidov merged 1 commit intomasterfrom
fix-too-large-allocation

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

@alexey-milovidov alexey-milovidov commented Dec 26, 2022

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)
SELECT
    range(arrayJoin(range(1048577)), 100),
    NULL
WHERE arrayJoin(range(255))
GROUP BY
    range(range(range(arrayJoin(range(7)), 1), range(10.0001, -2147483647, NULL)), arrayJoin(range(2147483646, arrayJoin(range(1048575)), -2147483648)), range(1023), arrayJoin(range(65537)), NULL),
    -1,
    range(3)
    WITH CUBE

std::vector was used where it shouldn't have been.

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Dec 26, 2022
@alexey-milovidov alexey-milovidov mentioned this pull request Dec 27, 2022
Merged
@alexey-milovidov
Copy link
Copy Markdown
Member Author

alexey-milovidov commented Dec 27, 2022

#44579

@alexey-milovidov alexey-milovidov self-assigned this Dec 27, 2022
@alexey-milovidov alexey-milovidov merged commit 279a27d into master Dec 27, 2022
@alexey-milovidov alexey-milovidov deleted the fix-too-large-allocation branch December 27, 2022 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@alexey-milovidov alexey-milovidov

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alexey-milovidov @robot-clickhouse