rdblue opened a new pull request, #5594: URL: https://github.com/apache/iceberg/pull/5594
The recent refactor of how properties work caused the CLI to stop working with REST catalogs that use tokens. The token was being fetched from properties using `self.property`, but was set in the constructor's local properties map. The two maps were different objects because properties are passed to `__init__` using `**properties`. The result was a 400 error (missing `Authorization` header). This also fixes the `_fetch_config` method's error handling. It was not parsing the error payload and so the cause message was ignored. This also updates text output to pretty print the stack trace when an exception causes the CLI to fail. -- 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]
