Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10524 )
Change subject: KUDU-2438 Class relocation in the maven build should be ...................................................................... Patch Set 2: (1 comment) I will check into the history of these changes today to bee sure but I think centralizing the relocation is a good idea and the issues that caused the change may be fixed now. http://gerrit.cloudera.org:8080/#/c/10524/2/java/pom.xml File java/pom.xml: http://gerrit.cloudera.org:8080/#/c/10524/2/java/pom.xml@179 PS2, Line 179: <relocations> Since these are all defined in the parent pom should they be removed from the child poms? -- To view, visit http://gerrit.cloudera.org:8080/10524 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib24a990544c58a46c08df673050543a67b0babf9 Gerrit-Change-Number: 10524 Gerrit-PatchSet: 2 Gerrit-Owner: Ferenc Szabo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Ferenc Szabo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Thu, 31 May 2018 15:43:27 +0000 Gerrit-HasComments: Yes
