HwangDongJun commented on PR #12831: URL: https://github.com/apache/gluten/pull/12831#issuecomment-6262518052
Hi @zhouyuan, gentle ping on this one. All CI checks are green now (x86, ARM, enhanced, format, license) after merging the current main, and I've updated the description with the verification results and known limitations. On extending this to Azure/GCS (your Aug 20 question): no SDK changes are needed, as both already expose a CA override (CAInfo/CAPath for Azure, CARootsFilePathOption for GCS). I don't have an Azure or GCS environment, though, so I could only compile-check those changes and not verify them at runtime. Would you prefer to merge the S3 part now and leave Azure/GCS as a follow-up, or would you still like me to add the untested change here? Thanks! -- 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]
