1

Author: HeartLinked

src/iceberg/test/set_snapshot_test.cc

@@ -19,8 +19,6 @@
 
 #include "iceberg/update/set_snapshot.h"
 
-#include <memory>
-
 #include <gmock/gmock.h>
 #include <gtest/gtest.h>
 
@@ -32,7 +30,6 @@
 
 namespace iceberg {
 
-// Test fixture for SetSnapshot tests
 class SetSnapshotTest : public UpdateTestBase {
  protected:
   // Snapshot IDs from TableMetadataV2Valid.json

src/iceberg/transaction.h

@@ -26,7 +26,6 @@
 #include "iceberg/iceberg_export.h"
 #include "iceberg/result.h"
 #include "iceberg/type_fwd.h"
-#include "iceberg/update/set_snapshot.h"
 
 namespace iceberg {
 
@@ -86,7 +85,7 @@ class ICEBERG_EXPORT Transaction : public std::enable_shared_from_this<Transacti
   /// \brief Create a new UpdateLocation to update the table location and commit the
   /// changes.
   Result<std::shared_ptr<UpdateLocation>> NewUpdateLocation();
-  
+
   /// \brief Create a new SetSnapshot to set the current snapshot or rollback to a
   /// previous snapshot and commit the changes.
   Result<std::shared_ptr<SetSnapshot>> NewSetSnapshot();

src/iceberg/util/snapshot_util.cc

@@ -104,8 +104,6 @@ Result<std::vector<std::shared_ptr<Snapshot>>> SnapshotUtil::CurrentAncestors(
 Result<std::vector<std::shared_ptr<Snapshot>>> SnapshotUtil::CurrentAncestors(
     const TableMetadata& metadata) {
   ICEBERG_ASSIGN_OR_RAISE(auto current, metadata.Snapshot());
-
-  // Create a lookup function that uses the metadata
   auto lookup = [&metadata](int64_t id) -> Result<std::shared_ptr<Snapshot>> {
     return metadata.SnapshotById(id);
   };