cnauroth commented on PR #8226: URL: https://github.com/apache/hadoop/pull/8226#issuecomment-3849493432
> Hi Steve. I tried applying your patch locally, but my build still didn't produce the expected results. > > However, I have a new idea. Take a look at #8147 , specifically: > > > 4. Reorder the module list in the top-level pom.xml so that hadoop-dist is always last. Otherwise, hadoop-dist runs the dist-layout-stitching script before hadoop-gcp has run its shading in the package phase, so it accidentally picks up the unshaded jar. > > Maybe what we need is a backport to branch-3.4 of just the root pom.xml change, so we make sure hadoop-cloud-storage-dist always runs before hadoop-dist. I'm testing this approach now and will report back after the build finishes. This worked for me. Can you please take a look at #8228 and let me know if that works for you? I didn't need to apply the changes from here. I just needed #8228. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
