Skip to content
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

Improve bazel builds #53

Merged
merged 21 commits into from
Jul 25, 2019
Merged

Improve bazel builds #53

merged 21 commits into from
Jul 25, 2019

Conversation

tkilias
Copy link
Collaborator

@tkilias tkilias commented Jul 24, 2019

This pull requests improves the bazel build

  • It improves the configuration of compiler flags to achieve similar compiler optimization for the udfclient than with the old build bash script
  • It activates link time optimization which might allow the compiler more optimizations than before
  • Removes unnecessary shared libraries for java, because these increased the loading time of the udfclient
  • It simplifies the rpath workaround by moving it to the java_repository.bzl
  • The build system can now build two variants of the executable, one with hard coded paths and one with a wrapper script which sets the paths via environment variables. The executable with hard coded paths only runs in the container environment, but has few percent faster startup than the wrapper script. The wrapper script can be used local debugging and testing outside of the container environment.

- reactivate binary wrapper for easier local testing and debugging in addition to rpath replacement
- explain rpath workarrounds
- structure bazelrc by adding config presets
- adjust the dockerfiles to new build presets
- add chrpath to build_deps packages
…ository and add start test for udfclient to build_run dockerfile
@tkilias tkilias merged commit dd54f66 into exasol:master Jul 25, 2019
@tkilias tkilias deleted the improve-bazel-builds branch July 25, 2019 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant