Revert "address comments"

This reverts commit 5e4a108.

Author: wgtmac

src/iceberg/table_metadata.cc

@@ -22,7 +22,6 @@
 #include <algorithm>
 #include <chrono>
 #include <format>
-#include <ranges>
 #include <string>
 
 #include <nlohmann/json.hpp>
@@ -185,37 +184,44 @@ Result<TableMetadataCache::SnapshotsMapRef> TableMetadataCache::GetSnapshotsById
 
 Result<TableMetadataCache::SchemasMap> TableMetadataCache::InitSchemasMap(
     const TableMetadata* metadata) {
-  return metadata->schemas | std::views::filter([](const auto& schema) {
-           return schema->schema_id().has_value();
-         }) |
-         std::views::transform([](const auto& schema) {
-           return std::pair(schema->schema_id().value(), schema);
-         }) |
-         std::ranges::to<SchemasMap>();
+  SchemasMap schemas_map;
+  schemas_map.reserve(metadata->schemas.size());
+  for (const auto& schema : metadata->schemas) {
+    if (schema->schema_id()) {
+      schemas_map.emplace(schema->schema_id().value(), schema);
+    }
+  }
+  return schemas_map;
 }
 
 Result<TableMetadataCache::PartitionSpecsMap> TableMetadataCache::InitPartitionSpecsMap(
     const TableMetadata* metadata) {
-  return metadata->partition_specs | std::views::transform([](const auto& spec) {
-           return std::pair(spec->spec_id(), spec);
-         }) |
-         std::ranges::to<PartitionSpecsMap>();
+  PartitionSpecsMap partition_specs_map;
+  partition_specs_map.reserve(metadata->partition_specs.size());
+  for (const auto& spec : metadata->partition_specs) {
+    partition_specs_map.emplace(spec->spec_id(), spec);
+  }
+  return partition_specs_map;
 }
 
 Result<TableMetadataCache::SortOrdersMap> TableMetadataCache::InitSortOrdersMap(
     const TableMetadata* metadata) {
-  return metadata->sort_orders | std::views::transform([](const auto& order) {
-           return std::pair(order->order_id(), order);
-         }) |
-         std::ranges::to<SortOrdersMap>();
+  SortOrdersMap sort_orders_map;
+  sort_orders_map.reserve(metadata->sort_orders.size());
+  for (const auto& order : metadata->sort_orders) {
+    sort_orders_map.emplace(order->order_id(), order);
+  }
+  return sort_orders_map;
 }
 
 Result<TableMetadataCache::SnapshotsMap> TableMetadataCache::InitSnapshotMap(
     const TableMetadata* metadata) {
-  return metadata->snapshots | std::views::transform([](const auto& snapshot) {
-           return std::pair(snapshot->snapshot_id, snapshot);
-         }) |
-         std::ranges::to<SnapshotsMap>();
+  SnapshotsMap snapshots_map;
+  snapshots_map.reserve(metadata->snapshots.size());
+  for (const auto& snapshot : metadata->snapshots) {
+    snapshots_map.emplace(snapshot->snapshot_id, snapshot);
+  }
+  return snapshots_map;
 }
 
 // TableMetadataUtil implementation