Skip to content

murmurhash32 ubsan fix.#7388

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
amosbird:ubunaligned
Oct 19, 2019
Merged

murmurhash32 ubsan fix.#7388
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
amosbird:ubunaligned

Conversation

@amosbird
Copy link
Copy Markdown
Collaborator

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Can be reproduced with ubsan build and following queries.

SELECT * from numbers(1000, 2) where murmurHash2_32(toString(number)) = 1;
SELECT * from numbers(1000, 2) where murmurHash3_32(toString(number)) = 1;

For changelog. Remove if this is non-significant change.

Category (leave one):

  • Bug Fix

@alexey-milovidov alexey-milovidov added the pr-build Pull request with build/testing/packaging improvement label Oct 19, 2019
@alexey-milovidov alexey-milovidov merged commit 2a9e2ab into ClickHouse:master Oct 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-build Pull request with build/testing/packaging improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants