Davis-Zhang-Onehouse commented on PR #18814: URL: https://github.com/apache/hudi/pull/18814#issuecomment-4744758743
@yihua > does the old release directly use the base path processed by s3aToS3? yes > One clarifying question, does the old release directly use the base path processed by s3aToS3? In most cases, the base path does not have a trailing slash. Adding a trailing slash from this PR means a behavior change different from the previous release. You make the call, and I want to keep you posted about 1 thing - in our production use case especially running spark code, it really depends on how user sets the table base path which could be either with or without a trailing "/". So no matter which route we choose, some of the behavior would get changed and not sure how much percentage spark code use case accounts for it. With this informed context, let me know if you prefer we standardize on without "/" or with "/". 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]
