zhztheplayer opened a new pull request, #8350: URL: https://github.com/apache/incubator-gluten/pull/8350
Remove `--enable_ep_cache` to simplify the build script. After the removal, `ccache` based build acceleration will be preferred. Reason: Overtime `--enable_ep_cache` brings more maintenance burden than its benefits. It was designed as a "smart" flag to let the build system decide whether to rebuild Velox code again, however ended up with a partial solution, which doesn't take all the build variables as the cache key. It's possible that we enhance the cache key generation code to make it consider all the variables however it will make the build script more complicated. The patch removes `--enable_ep_cache` for the reason. `ccache` based build acceleration will be preferred. In future we could add a manual flag or a command line prompt to skip Velox build if similar functionality is still needed. -- 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]
