-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Fiddle with various build options #309
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
Comments
I ran some tests on some of these options, specifically Setting
Unfortunately, it comes at a great expense of compile time. On a 8 CU (4 core, 2 thread) EC2 system, turning both options on doubles the wall time taken (though amusingly the CPU time drops). The proportion of the time increase would grow larger at higher core counts due to the reduced parallelism.
Both enabling LTO and reducing |
@bruceg These tests look great! We have "fat" Rust codegen LTO enabled for building our release binaries and run For some time we also had "fat" LLVM cross-language LTO enabled (see #689 for some comparisons of Vector binary size), but it was switched off as part of #794. Because now Also it is interesting to consider compressing Vector with UPX (see #1797 for details). |
OK, the performance numbers. These are geometric means of the median time across all 29 benchmarks:
So, turning on LTO has a significant impact on performance. Setting |
…rser (vectordotdev#309) 1. Change apache error log date format to the common format 2. Add module identifier to mezmo apache demo log Ref: LOG-17701
## [1.6.3](answerbook/vector@v1.6.2...v1.6.3) (2023-08-10) ### Bug Fixes * **mezmo logs**: Make apache demo error logs parseable by pipeline parser (vectordotdev#309) [615572a](answerbook/vector@615572a) - GitHub [LOG-17701](https://logdna.atlassian.net/browse/LOG-17701)
These aren't all necessarily a good idea to ship, but they should have some impact on size or speed or the resulting binary that's worth measuring.
panic = "abort"
(I don't think we want to do this since we being able to clean up after a panic is a thing we want, but it'd still be nice to measure the impact.)strip
ing debug symbolscodegen-units = 1
The text was updated successfully, but these errors were encountered: