anujmodi2021 commented on PR #7252: URL: https://github.com/apache/hadoop/pull/7252#issuecomment-2572328303
Thanks for the PR @horizonzy We have a few things that we want to understand better. Can you please update the description with proper explanation of requirement and how this code change helping? 1. Why was this change made on wasb driver? [Wasb is on a path of deprecation](https://issues.apache.org/jira/browse/HADOOP-19178) and we already support 2. Why do we need any other URL when Azurite is supposed to run on localhost always? 3. What does "another container" means here? Do you want to work with multiple filesystems? @steveloughran we already have support for azurite emulator on ABFS. It might need better documentation though, will chcek and work on that. But for CI and PR sign offs we would still prefer testing using proper production accounts due to following reasons. 1. Azure storage services are rapidly changing including/updating a lot of features. We are not sure how often azurite is updated to reflect those changes. 2. Any release and update of azurite will force us to update the CI setup, that will be an additional manual maintainence. There is a value in terms of saving time during local dev and there we can consider using this more often. -- 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]
