please don't delete AwsHostNameUtils
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 2.6k
- Forks
- 1k
- Ø Merge
- 2 T. 9 Std.
- Gemergte PRs (30 T.)
- 51
Beschreibung
Describe the feature
I've noticed that AwsHostNameUtils has a comment "TODO This isn't used for as many services as it supports. Can we simplify or remove it?".
It is used in the hadoop s3a client to work out the signing region of a bucket.
Removing it would break things and make people sad. 😢 (*)
Please leave, and ideally update the docs to say "this is used externally". Thanks.
(*) also it'd stop them dropping in updated SDK releases to hadoop installations and lead to bug reports on this project
Use Case
getS3RegionFromEndpoint() to continue to return region endpoints.
Proposed Solution
No response
Other Information
No response
Acknowledgements
- I may be able to implement this feature request
- This feature might incur a breaking change
AWS Java SDK version used
2.35.4
JDK version used
8, 17
Operating System and version
various
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Finde AwsHostNameUtils und seinen Einstiegspunkt getS3RegionFromEndpoint(), und lies das vorhandene TODO sowie die umgebende Dokumentation. Bestätige die im Issue beschriebene Verwendung durch Hadoop S3A und aktualisiere die relevante Dokumentation, um festzuhalten, dass das Dienstprogramm extern verwendet wird; als erledigt gilt die Aufgabe, wenn das Dienstprogramm weiterhin verfügbar ist und seine externe Verwendung dokumentiert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- aws, java
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100