Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/23407 )
Change subject: KUDU-3697: Bundle kudu-spark classes ...................................................................... Patch Set 1: Correcting the formatting: > curiosity question: how did you notice this regression? This was discovered in Cloudera test suite during integration testing. All the tests which relied on the published jars (which involved kudu-spark) connector failed. > I'm curious about this as well. Also which classes are missing from the jars? All the classes from kudu-spark module are missing from the published kudu-backup and kudu-spark-tools jar. > Have you tried disabling the shadowJar minimising? Yes, that did not help. >I think it's always beneficial to have more tools/procedures/guardrails to >spot issues like this, especially in the context of the series of fallout >after the recent Gradle 7.6.4 upgrade. It would be nice to add corresponding script or at least manual steps into RELEASING.adoc, making sure we aren't hitting the same issue again in future releases (if any). Consider doing that in a separate patch, or at least file a JIRA with all necessary information provided. Agree, I think we should have some tests/scripts which run against the published jars to catch any regression. I'll raise a corresponding Jira. -- To view, visit http://gerrit.cloudera.org:8080/23407 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8d3e3c95efe161adc916332477f8cd9f35c48e38 Gerrit-Change-Number: 23407 Gerrit-PatchSet: 1 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Wed, 10 Sep 2025 20:51:41 +0000 Gerrit-HasComments: No
