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
AVX2 Support #7
Comments
Thank you for your suggestion. Before, I just thought that most Linux servers should have AVX2 support for performance considerations. Didn't realize usage scenarios like lambdas, virtualized cpus. I'm going to fix this in v0.5.0. |
Thanks! If you would like to support AWS Lambdas and other virtualized system the following patch is also required:
Would you accept PRs to the builder? |
PR is welcome. |
The strip issue is mainly because currently chdb is really new. I need debug info if any crash happens. |
That makes sense, although the resulting file is really huge.
after
|
PR open, moving discussion in thread: #9 |
Great mod and very interesting binding approach! We're also playing with clickhouse-local custom builds towards a slimmer execution engine.
Question: Is there a specific reason for AVX2 to be enabled for Linux systems? This greatly reduces the scope of usage (ie: lambdas, virtualized cpus, etc, fail with
Instruction check fail. The CPU does not support AVX2 instruction set.
)Having this disabled (or possibly multiple versions) could broaden compatibility scope enormously. Also the library doesn't seem to be stripped and the distributed .so has 100M+ of trimmable fat.
Thanks!
The text was updated successfully, but these errors were encountered: