wolflex888 commented on PR #12595: URL: https://github.com/apache/iceberg/pull/12595#issuecomment-2743778461
> @wolflex888 thank you so much for reporting this. > > I was initially surprised because this functionality was supposed to work just as before. After digging a bit, I found the culprit: in this commit I inadvertently removed one line that should not have been removed: > > [77a25fa#diff-86450612dbe323d6d06cbc3846aa1913f042eaedadc0ca027c36bfbe08d3a46cL174](https://github.com/apache/iceberg/commit/77a25fa643c610978e27d94a8d943811c91eed2e#diff-86450612dbe323d6d06cbc3846aa1913f042eaedadc0ca027c36bfbe08d3a46cL174) > > I think the proper "fix" would be to just put that line back. For example: > > ```java > public RESTSessionCatalog() { > this( > config -> { > HTTPClient.Builder builder = > HTTPClient.builder(config).uri(config.get(CatalogProperties.URI)); > configHeaders(config).forEach(builder::withHeader); > return builder.build(); > }, > null); > } > ``` > > I am sorry for the inconvenience. Let me test that a bit. I think i did something similar and it did not work as expected.. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
