Re: [PR] HTTPRequest: don't remove trailing slash from absolute paths [iceberg]

2025-02-23 Thread via GitHub
ajantha-bhat commented on PR #12389: URL: https://github.com/apache/iceberg/pull/12389#issuecomment-2676845372 @adutra: Thanks for the quick fix. Once, nit is handled, please raise the same PR against branch 1.8.x https://github.com/apache/iceberg/tree/1.8.x -- This is an automated mes

Re: [PR] HTTPRequest: don't remove trailing slash from absolute paths [iceberg]

2025-02-23 Thread via GitHub
adutra commented on PR #12389: URL: https://github.com/apache/iceberg/pull/12389#issuecomment-2676847699 @ajantha-bhat here is the 1.8.x PR: #12390 -- 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 t

Re: [PR] HTTPRequest: don't remove trailing slash from absolute paths [iceberg]

2025-02-23 Thread via GitHub
ajantha-bhat commented on code in PR #12389: URL: https://github.com/apache/iceberg/pull/12389#discussion_r1966767782 ## core/src/main/java/org/apache/iceberg/rest/HTTPRequest.java: ## @@ -53,13 +53,16 @@ enum HTTPMethod { */ @Value.Lazy default URI requestUri() { -