Skip to content

Fix CPU-dependent behaviour in tests#22590

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-tests-aarch64
Apr 4, 2021
Merged

Fix CPU-dependent behaviour in tests#22590
alexey-milovidov merged 1 commit intomasterfrom
fix-tests-aarch64

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Adjust some tests to output identical results on amd64 and aarch64 (qemu). The result was depending on implementation specific CPU behaviour.

Details:

$ clickhouse-local --query "SELECT toInt32(72004899337.38588)"
-2147483648
$ qemu-aarch64-static clickhouse-aarch64-2 local --query "SELECT toInt32(72004899337.38588)"
<jemalloc>: MADV_DONTNEED does not work (memset will be used instead)
<jemalloc>: (This is the expected behaviour if you are running under QEMU)
2147483647

@robot-clickhouse robot-clickhouse added the pr-build Pull request with build/testing/packaging improvement label Apr 4, 2021
@alexey-milovidov alexey-milovidov self-assigned this Apr 4, 2021
@alexey-milovidov alexey-milovidov merged commit f6e6842 into master Apr 4, 2021
@alexey-milovidov alexey-milovidov deleted the fix-tests-aarch64 branch April 4, 2021 12:43
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