Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/24583 )
Change subject: KUDU-3788 an option to use pre-built thirdparty artifacts ...................................................................... Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/24583/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/24583/5//COMMIT_MSG@30 PS5, Line 30: After that, it's : possible to upload the result into a sub-directory of the designated : S3 bucket s3://cloudera-thirdparty-libs/prebuilt Question: To leverage this gain, how do we manage the manual upload of pre-built artifacts to S3, given there would be a good number of archives for even a limited and usual configuration/setup (just from CI) combinatorial space? http://gerrit.cloudera.org:8080/#/c/24583/5//COMMIT_MSG@35 PS5, Line 35: distrute distribute http://gerrit.cloudera.org:8080/#/c/24583/5//COMMIT_MSG@45 PS5, Line 45: uninstrumeted uninstrumented http://gerrit.cloudera.org:8080/#/c/24583/5//COMMIT_MSG@45 PS5, Line 45: /opt/kudu/ Have you considered 'home' directory approach to avoid any first-time directory creation (/opt/kudu/thirdparty) and permission setting challenges, etc. ? I am guessing single user CI/dev system is the general use case here and concurrent build by multiple users is not something that needs to be considered here? http://gerrit.cloudera.org:8080/#/c/24583/5//COMMIT_MSG@62 PS5, Line 62: Only CMake/GNUmake combination is now used for building : 3rd-party components that rely on CMake to bootstrap build environment Is there a significant increase in build time if not using ninja, especially for LLVM? I understand that this is supposed to be one-time penalty, considering artifacts would get built and uploaded to S3 in timely fashion. As a result, subsequent builds would download and use those directly, thereby saving time but just for curiosity. -- To view, visit http://gerrit.cloudera.org:8080/24583 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I55b5b99fcbe60eec3f6ae19fa4f2e335f361d143 Gerrit-Change-Number: 24583 Gerrit-PatchSet: 5 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: Joe McDonnell <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Comment-Date: Tue, 28 Jul 2026 08:46:05 +0000 Gerrit-HasComments: Yes
