kbendick opened a new pull request, #4636: URL: https://github.com/apache/iceberg/pull/4636
There is now a global singleton object mapper for the `RESCatalog`, which already has the necessary serializers and deserializers registered. The rest / response objects are all tested via the `RequestResponseTestBase` class, which currently repeats the object mapper instantiation and initialization logic that the `RESTObjectMapper` is ultimately responsible for. We should be testing the rest objects using the correct object mapper (which should also reduce allocations during testing and other minor advantages). cc @rdblue @danielcweeks -- 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]
