fix comments

Author: xiao.dong

src/iceberg/avro/avro_register.cc

@@ -19,11 +19,18 @@
 
 #include "avro_register.h"
 
+#include "iceberg/avro/avro_schema_util_internal.h"
+
 namespace iceberg::avro {
 
 void RegisterLogicalTypes() {
-  ::avro::CustomLogicalTypeRegistry::instance().registerType(
-      "map", [](const std::string&) { return std::make_shared<MapLogicalType>(); });
+  static std::once_flag flag{};
+  std::call_once(flag, []() {
+    // Register the map logical type with the avro custom logical type registry.
+    // See https://github.com/apache/avro/pull/3326 for details.
+    ::avro::CustomLogicalTypeRegistry::instance().registerType(
+        "map", [](const std::string&) { return std::make_shared<MapLogicalType>(); });
+  });
 }
 
 }  // namespace iceberg::avro

src/iceberg/avro/avro_register.h

@@ -19,14 +19,10 @@
 
 #pragma once
 
-#include <avro/LogicalType.hh>
+#include "iceberg/iceberg_bundle_export.h"
 
 namespace iceberg::avro {
 
-struct MapLogicalType : public ::avro::CustomLogicalType {
-  MapLogicalType() : ::avro::CustomLogicalType("map") {}
-};
-
-void RegisterLogicalTypes();
+ICEBERG_BUNDLE_EXPORT void RegisterLogicalTypes();
 
 }  // namespace iceberg::avro

src/iceberg/avro/avro_schema_util.cc

@@ -50,12 +50,7 @@ constexpr std::string_view kElementIdProp = "element-id";
 constexpr std::string_view kAdjustToUtcProp = "adjust-to-utc";
 
 ::avro::LogicalType GetMapLogicalType() {
-  static std::once_flag flag{};
-  std::call_once(flag, []() {
-    // Register the map logical type with the avro custom logical type registry.
-    // See https://github.com/apache/avro/pull/3326 for details.
-    RegisterLogicalTypes();
-  });
+  RegisterLogicalTypes();
   return ::avro::LogicalType(std::make_shared<MapLogicalType>());
 }
 

src/iceberg/avro/avro_schema_util_internal.h

@@ -34,6 +34,10 @@ class ValidSchema;
 
 namespace iceberg::avro {
 
+struct MapLogicalType : public ::avro::CustomLogicalType {
+  MapLogicalType() : ::avro::CustomLogicalType("map") {}
+};
+
 /// \brief A visitor that converts an Iceberg type to an Avro node.
 class ToAvroNodeVisitor {
  public: