DanielLeens commented on issue #11655: URL: https://github.com/apache/seatunnel/issues/11655#issuecomment-5210721977
Thanks for following up with PR #11673 and for keeping the issue scoped to the assumed-role S3A path instead of mixing it back into #11648. The current direction still looks correct to me: this is a real shading/packaging bug on the runtime classpath boundary, and the safest repair is the one that makes `seatunnel-hadoop-aws` agree with the relocation strategy already used by the Hadoop uber jar, rather than broadening the classpath surface unnecessarily. Keeping the issue open for tracking is fine, but the implementation discussion should now continue primarily on PR #11673. From the issue side, the remaining checks are mainly: 1. the assumed-role `RoleModel` / `JsonSerialization` path works on the shaded artifact at runtime; 2. the relocation does not break other Hadoop AWS loading paths; 3. CI stays green on the current PR head. Since you marked that you are willing to submit the fix, I've assigned the issue to you. -- 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]
