bogthe opened a new pull request #3516: URL: https://github.com/apache/hadoop/pull/3516
### Description of PR Turns out there was a small issue with the previous implementation that always returned `false` for `verifyBucketExistsV2`. The more "clearer" way to check is for an error message when there's a 403. If that error message is present then the AP doesn't exist (the only case when a 403 is returned for an AP), otherwise 404 means definite no. ### How was this patch tested? Ran `mvn -Dparallel-tests -DtestsThreadCount=32 clean verify` on `eu-west-1` with access points enabled (i.e. bucket pointing to AP ARN). ### For code changes: - [ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')? - [ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation? - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)? - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, `NOTICE-binary` files? -- 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]
