Hello Marton Greber, Zoltan Martonka, Ashwani Raina, Kudu Jenkins, Abhishek
Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24710
to look at the new patch set (#8).
Change subject: KUDU-3788 introduce 'any' for OS, OS version, arch, toolchain
......................................................................
KUDU-3788 introduce 'any' for OS, OS version, arch, toolchain
Some of the 3rd-party artifacts used by Kudu are architecture and/or
OS-independent: e.g., Java JARs, JS scripts, Python code, etc.
It doesn't make much sense to store same bits duplicated in pre-built
archives that differ only by name, especially if those
OS/arch-independent pre-built artifacts are the heaviest ones
(e.g., that's so for JARs).
This changelist introduces 'any' as a wildcard for matching against OS,
OS version, architecture, and the build toolchain. Enabling the 'any'
wildcard is done by setting corresponding <comp>_ANY_OS,
<comp>_OS_VERSION, <comp>_ANY_ARCH, and <comp>_ANY_TOOLCHAIN variables
to non-zero in thirdparty/vars.sh. This changelist does so for hadoop,
hive, ranger, ranger_kms, postgres_jdbcspace space hogs and for the
rest of the components that don't build any binaries and don't modify
the contents of their source tarballs: trace-viewer, gcovr, sparsepp,
sparsehash, cpplint, rapidjson. This is also done for jwt-cpp: it's
a header-only library with a non-trivial build procedure to find proper
SSL library implementation to build against.
Prometheus is marked with PROMETHEUS_ANY_OS_VERSION and
PROMETHEUS_ANY_TOOLCHAIN. With that, there is still duplication in
pre-built artifacts for different Linux flavors/distros. A follow-up
changelist may take care of this, perhaps introducing variables such as
<comp>_ANY_LINUX_DISTRO, <comp>_ANY_OS_FLAVOR, or similar.
Change-Id: I52428b8f2a558feb4d8247f37351ce2b6df1e4f7
---
M thirdparty/fetch-utils.sh
M thirdparty/prebuilt-utils.sh
M thirdparty/vars.sh
3 files changed, 98 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/10/24710/8
--
To view, visit http://gerrit.cloudera.org:8080/24710
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I52428b8f2a558feb4d8247f37351ce2b6df1e4f7
Gerrit-Change-Number: 24710
Gerrit-PatchSet: 8
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
