Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15360 )
Change subject: [build] Use Gradle to build the hms-plugin jar ...................................................................... Patch Set 2: > We can leave this if it's preferred. I am not strongly opinionated about it. No, I think we should either use add_jar() for all subordinate Java components, or gradle. Mixing and matching both is just more confusing. > If speed is a concern on jenkins, I can move the Java build above the c++ > build and it should look instant in the c++ build given Gradle will already > be primed with all the deps and skip all tasks. If it's a concern for local > builds, it should only be slower the first time. And hopefully not much. There's just a higher fixed cost to starting gradle than there is to using add_jar(). But like I told Andrew earlier, I'm OK with trying out this approach; we can always reverse course if we don't like it. -- To view, visit http://gerrit.cloudera.org:8080/15360 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3312b1605c4f050380264688f9d1b78038608257 Gerrit-Change-Number: 15360 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 04 Mar 2020 17:52:28 +0000 Gerrit-HasComments: No
