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
Implement support for different UUID binary formats #42108
Implement support for different UUID binary formats #42108
Conversation
0fcf42b
to
c037886
Compare
I checked the failing builds and I can see from the ones I could interpret that the constructor of |
a5323e2
to
208d992
Compare
pls check ASTFuzzer error report |
I'll make sure to do so, thanks! |
@nickitat I appreciate you taking the time to review my changes. I've updated the pull request and I believe that the issue detected by the fuzzer should be resolved. |
66ffd55
to
3f6b563
Compare
if (arguments.size() < 2) | ||
return UUIDSerializer::Variant::Default; | ||
|
||
const auto representation = static_cast<magic_enum::underlying_type_t<UUIDSerializer::Variant>>(arguments[1].column->getInt(0)); |
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.
arguments[1].column->getInt(0)
AFAIK, the column may be empty (especially when called from executeDryRunImpl
), so it may crash:
https://s3.amazonaws.com/clickhouse-test-reports/61799/ace983f3d5d20c4cfaf6220cda7e3251bfb93278/ast_fuzzer__msan_.html
msan report: https://pastila.nl/?000d758a/f6f27b43f714b735c5b0bb5246f3531c#DdJW2oeYhxwNOiJON6ioOQ==
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Implement support for different UUID binary formats with support for the two most prevalent ones: the default big-endian and Microsoft's mixed-endian as specified in RFC 4122