mbutrovich commented on issue #6210: URL: https://github.com/apache/datafusion-comet/issues/6210#issuecomment-5838761742
> 3. **Remove `lto = "thin"` from the release profile.** The profile then says what actually gets built, and build times stay as they are, giving up the Q21 gain. I think this is a good investigation, but I don't think LTO is the high pole in the tent from an optimization standpoint. I'd push for the third option just to make it clear what we're actually doing, and document why LTO isn't there with a link to this issue that it's not doing anything anyway. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
