Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23921 )
Change subject: [build] Include OS version in kudu-binary artifact names ...................................................................... Patch Set 2: Code-Review+1 > Overall looks good to me, just a couple questions: > -was there any particular issue that came up, that is fixed by this > change? > -does the Kudu Java example work as expected with the new naming? > (just to make sure that what we have in the example pom, works) +1 How do clients know what JAR to fetch with this change in the name? Is it necessary to update corresponding examples, etc. along with this update? -- To view, visit http://gerrit.cloudera.org:8080/23921 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7f17971704dfec32a0256383bbc3029743b4674f Gerrit-Change-Number: 23921 Gerrit-PatchSet: 2 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Comment-Date: Wed, 04 Feb 2026 22:29:30 +0000 Gerrit-HasComments: No
