1

Author: HeartLinked

src/iceberg/schema.cc

@@ -27,7 +27,6 @@
 #include "iceberg/schema_internal.h"
 #include "iceberg/type.h"
 #include "iceberg/util/formatter.h"  // IWYU pragma: keep
-#include "iceberg/util/formatter_internal.h"
 #include "iceberg/util/macros.h"
 #include "iceberg/util/type_util.h"
 #include "iceberg/util/visit_type.h"

src/iceberg/util/type_util.cc

@@ -26,8 +26,6 @@
 
 namespace iceberg {
 
-// IdToFieldVisitor implementation
-
 IdToFieldVisitor::IdToFieldVisitor(
     std::unordered_map<int32_t, std::reference_wrapper<const SchemaField>>& id_to_field)
     : id_to_field_(id_to_field) {}
@@ -47,8 +45,6 @@ Status IdToFieldVisitor::Visit(const NestedType& type) {
   return {};
 }
 
-// NameToIdVisitor implementation
-
 NameToIdVisitor::NameToIdVisitor(
     std::unordered_map<std::string, int32_t, StringHash, std::equal_to<>>& name_to_id,
     bool case_sensitive, std::function<std::string(std::string_view)> quoting_func)

src/iceberg/util/type_util.h

@@ -42,7 +42,6 @@
 namespace iceberg {
 
 /// \brief Visitor for building a map from field ID to SchemaField reference.
-/// Corresponds to Java's IndexById visitor.
 class IdToFieldVisitor {
  public:
   explicit IdToFieldVisitor(
@@ -56,7 +55,6 @@ class IdToFieldVisitor {
 };
 
 /// \brief Visitor for building a map from field name to field ID.
-/// Corresponds to Java's IndexByName visitor.
 class NameToIdVisitor {
  public:
   explicit NameToIdVisitor(
@@ -85,7 +83,6 @@ class NameToIdVisitor {
 };
 
 /// \brief Visitor for building a map from field ID to position path.
-/// Used for efficient field access in StructLike.
 class PositionPathVisitor {
  public:
   Status Visit(const PrimitiveType& type) {
@@ -129,7 +126,6 @@ class PositionPathVisitor {
 };
 
 /// \brief Visitor for pruning columns based on selected field IDs.
-/// Corresponds to Java's PruneColumns visitor.
 ///
 /// This visitor traverses a schema and creates a projected version containing only
 /// the specified fields. When `select_full_types` is true, a field with all its