feat(rest): build resource paths after get config.

Author: plusplusjiajia

src/iceberg/catalog/rest/resource_paths.cc

@@ -46,8 +46,8 @@ Status ResourcePaths::SetBaseUri(const std::string& base_uri) {
   return {};
 }
 
-Result<std::string> ResourcePaths::Config() const {
-  return std::format("{}/v1/config", base_uri_);
+Result<std::string> ResourcePaths::Config(const std::string& base_uri) {
+  return std::format("{}/v1/config", base_uri);
 }
 
 Result<std::string> ResourcePaths::OAuth2Tokens() const {

src/iceberg/catalog/rest/resource_paths.h

@@ -23,7 +23,6 @@
 #include <string>
 
 #include "iceberg/catalog/rest/iceberg_rest_export.h"
-#include "iceberg/catalog/rest/type_fwd.h"
 #include "iceberg/result.h"
 #include "iceberg/type_fwd.h"
 
@@ -48,7 +47,7 @@ class ICEBERG_REST_EXPORT ResourcePaths {
   Status SetBaseUri(const std::string& base_uri);
 
   /// \brief Get the /v1/config endpoint path.
-  Result<std::string> Config() const;
+  static Result<std::string> Config(const std::string& base_uri);
 
   /// \brief Get the /v1/{prefix}/oauth/tokens endpoint path.
   Result<std::string> OAuth2Tokens() const;

src/iceberg/catalog/rest/rest_catalog.cc

@@ -63,9 +63,8 @@ std::unordered_set<Endpoint> GetDefaultEndpoints() {
 }
 
 /// \brief Fetch server config and merge it with client config
-Result<CatalogConfig> FetchServerConfig(const ResourcePaths& paths,
+Result<CatalogConfig> FetchServerConfig(const std::string& config_path,
                                         const RestCatalogProperties& current_config) {
-  ICEBERG_ASSIGN_OR_RAISE(auto config_path, paths.Config());
   HttpClient client(current_config.ExtractHeaders());
   ICEBERG_ASSIGN_OR_RAISE(const auto response,
                           client.Get(config_path, /*params=*/{}, /*headers=*/{},
@@ -111,10 +110,9 @@ Result<std::shared_ptr<RestCatalog>> RestCatalog::Make(
   if (!file_io) {
     return InvalidArgument("FileIO is required to create RestCatalog");
   }
-  ICEBERG_ASSIGN_OR_RAISE(
-      auto paths, ResourcePaths::Make(std::string(TrimTrailingSlash(uri)),
-                                      config.Get(RestCatalogProperties::kPrefix)));
-  ICEBERG_ASSIGN_OR_RAISE(auto server_config, FetchServerConfig(*paths, config));
+  ICEBERG_ASSIGN_OR_RAISE(auto config_uri,
+                          ResourcePaths::Config(std::string(TrimTrailingSlash(uri))));
+  ICEBERG_ASSIGN_OR_RAISE(auto server_config, FetchServerConfig(config_uri, config));
 
   std::unique_ptr<RestCatalogProperties> final_config = RestCatalogProperties::FromMap(
       MergeConfigs(server_config.defaults, config.configs(), server_config.overrides));
@@ -132,7 +130,9 @@ Result<std::shared_ptr<RestCatalog>> RestCatalog::Make(
 
   // Update resource paths based on the final config
   ICEBERG_ASSIGN_OR_RAISE(auto final_uri, final_config->Uri());
-  ICEBERG_RETURN_UNEXPECTED(paths->SetBaseUri(std::string(TrimTrailingSlash(final_uri))));
+  ICEBERG_ASSIGN_OR_RAISE(
+      auto paths, ResourcePaths::Make(std::string(TrimTrailingSlash(final_uri)),
+                                      final_config->Get(RestCatalogProperties::kPrefix)));
 
   return std::shared_ptr<RestCatalog>(
       new RestCatalog(std::move(final_config), std::move(file_io), std::move(paths),