ryanvflannery commented on PR #10962: URL: https://github.com/apache/seatunnel/pull/10962#issuecomment-4646427189
Addressed the review feedback: - Added the missing ASF license header. - Moved Azure Key Vault ConfigShade into a dedicated optional module. - Removed Azure SDK dependencies from seatunnel-core-starter. - Packaged the optional provider with its Azure runtime dependencies. - Added regression tests for ServiceLoader discovery and secret resolution. - Added English and Chinese documentation for azure-kv configuration. Verification completed: - Spotless passed. - All 5 focused Azure provider tests passed. - Optional provider packaging passed. - Confirmed the packaged JAR contains the provider, Azure SDK classes, and ConfigShade SPI descriptor. Full verification was also attempted locally, but Windows shaded-JAR file-locking errors prevented completion; Linux CI will provide the final full-build result. -- 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]
