feat: Refresh method support for table

Author: shuxu.li

src/iceberg/catalog/in_memory_catalog.cc

@@ -109,15 +109,6 @@ class ICEBERG_EXPORT InMemoryNamespace {
   ///         ErrorKind::kNoSuchTable if the table does not exist.
   Status UnregisterTable(const TableIdentifier& table_ident);
 
-  /// \brief Updates the metadata location of an existing table.
-  ///
-  /// \param table_ident The fully qualified identifier of the table.
-  /// \param metadata_location The path to the table's metadata.
-  /// \return Status::OK if the table metadata location is updated;
-  ///         Error otherwise.
-  Status UpdateTableMetaLocation(const TableIdentifier& table_ident,
-                                 const std::string& metadata_location);
-
   /// \brief Checks if a table exists in the specified namespace.
   ///
   /// \param table_ident The identifier of the table to check.
@@ -306,17 +297,6 @@ Status InMemoryNamespace::UnregisterTable(TableIdentifier const& table_ident) {
   return {};
 }
 
-Status InMemoryNamespace::UpdateTableMetaLocation(const TableIdentifier& table_ident,
-                                                  const std::string& metadata_location) {
-  const auto ns = GetNamespace(this, table_ident.ns);
-  ICEBERG_RETURN_UNEXPECTED(ns);
-  if (!ns.value()->table_metadata_locations_.contains(table_ident.name)) {
-    return NotFound("{} does not exist", table_ident.name);
-  }
-  ns.value()->table_metadata_locations_[table_ident.name] = metadata_location;
-  return {};
-}
-
 Result<bool> InMemoryNamespace::TableExists(TableIdentifier const& table_ident) const {
   const auto ns = GetNamespace(this, table_ident.ns);
   ICEBERG_RETURN_UNEXPECTED(ns);

test/in_memory_catalog_test.cc

@@ -151,7 +151,7 @@ TEST_F(InMemoryCatalogTest, RefreshTable) {
   ASSERT_TRUE(table.value()->current_snapshot().has_value());
   ASSERT_EQ(table.value()->current_snapshot().value()->snapshot_id, 3051729675574597004);
 
-  // same version
+  // refresh table to new snapshot
   auto status = table.value()->Refresh();
   EXPECT_THAT(status, IsOk());
   ASSERT_TRUE(table.value()->current_snapshot().has_value());

test/mock_catalog.h

@@ -23,7 +23,7 @@
 
 namespace iceberg {
 
-class ICEBERG_EXPORT MockInMemoryCatalog : public InMemoryCatalog {
+class MockInMemoryCatalog : public InMemoryCatalog {
  public:
   MockInMemoryCatalog(std::string const& name, std::shared_ptr<FileIO> const& file_io,
                       std::string const& warehouse_location,