feat: static table metadata access support

Author: shuxu.li

src/iceberg/table.cc

@@ -146,7 +146,7 @@ const std::vector<std::shared_ptr<Snapshot>>& BaseTable::snapshots() const {
 }
 
 const std::vector<std::shared_ptr<HistoryEntry>>& BaseTable::history() const {
-  // TODO: Implement history retrieval
+  // TODO(lishuxu): Implement history retrieval
   throw IcebergError("history is not supported for BaseTable now");
 }
 

src/iceberg/table.h

@@ -111,7 +111,7 @@ class ICEBERG_EXPORT Table {
 
 class ICEBERG_EXPORT BaseTable : public Table {
  public:
-  virtual ~BaseTable() override = default;
+  ~BaseTable() override = default;
   BaseTable(std::string name, std::shared_ptr<TableMetadata> metadata);
 
   const std::string& name() const override { return name_; }
@@ -175,7 +175,7 @@ class ICEBERG_EXPORT BaseTable : public Table {
 
 class ICEBERG_EXPORT StaticTable : public BaseTable {
  public:
-  virtual ~StaticTable() override = default;
+  ~StaticTable() override = default;
   StaticTable(std::string name, std::shared_ptr<TableMetadata> metadata)
       : BaseTable(std::move(name), std::move(metadata)) {}
 

test/table_test_helper.h

@@ -28,10 +28,13 @@ namespace iceberg {
 
 class TableTestHelper {
  public:
+  /// \brief Get the full path to a resource file in the test resources directory
   static std::string GetResourcePath(const std::string& file_name);
 
+  /// \brief Read a JSON file from the test resources directory
   static void ReadJsonFile(const std::string& file_name, std::string* content);
 
+  /// \brief Read table metadata from a JSON file in the test resources directory
   static void ReadTableMetadata(const std::string& file_name,
                                 std::unique_ptr<TableMetadata>* metadata);
 };