shrutisinghania commented on PR #52027: URL: https://github.com/apache/spark/pull/52027#issuecomment-3276242769
Thank you for your detailed feedback and valuable suggestions, @dongjoon-hyun. I have updated the pull request to address the concerns you raised. Specifically: * The logic now checks for the presence of the GCS connector on the classpath, so there is no performance regression for non-GCS users. * The prefixing logic is now idempotent and checks if the prefix already exists, which prevents duplication I have also updated the PR description to provide a clearer explanation of the benefits and to include a concrete example, as you suggested. Could you please take another look when you have a moment? Thanks again for your guidance -- 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]
