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
Fix crash in median over Nullable(Decimal128) #6378
Conversation
There's no alignment in AllocatorWithStackMemory. It leads to crash in attepmt to push_back __int128 into PODArrayWithStackMemory |
Also we can replace |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Asked a question in Telegram chat.
dbms/src/Common/PODArray.h
Outdated
@@ -636,6 +636,6 @@ using PaddedPODArray = PODArray<T, initial_bytes, TAllocator, 15, 16>; | |||
template <typename T, size_t inline_bytes, | |||
size_t rounded_bytes = integerRoundUp(inline_bytes, sizeof(T))> | |||
using PODArrayWithStackMemory = PODArray<T, rounded_bytes, | |||
AllocatorWithStackMemory<Allocator<false>, rounded_bytes>>; | |||
AllocatorWithStackMemory<Allocator<false>, rounded_bytes, ((sizeof(T) < 16) ? sizeof(T) : 16)>>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is also alignof
keyword.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll change.
Fix crash in median over Nullable(Decimal128) (cherry picked from commit b579692)
Fix crash in median over Nullable(Decimal128) (cherry picked from commit b579692)
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Category (leave one):
Short description (up to few sentences):
Crash in median over Nullable(Decimal128)