remove kMetadata

wgtmac · wgtmac · commit e3ccc4f36400 · 2025-04-29T16:02:22.000+08:00
diff --git a/src/iceberg/file_format.h b/src/iceberg/file_format.h
@@ -34,7 +34,6 @@ enum class ICEBERG_EXPORT FileFormatType {
   kAvro,
   kOrc,
   kPuffin,
-  kMetadata,
 };
 
 /// \brief Convert a FileFormatType to a string
@@ -48,8 +47,6 @@ ICEBERG_EXPORT inline std::string_view ToString(FileFormatType format_type) {
       return "orc";
     case FileFormatType::kPuffin:
       return "puffin";
-    case FileFormatType::kMetadata:
-      return "metadata";
   }
 }
 
diff --git a/src/iceberg/file_reader.cc b/src/iceberg/file_reader.cc
@@ -42,7 +42,6 @@ ReaderFactory& ReaderFactoryRegistry::GetFactory(FileFormatType format_type) {
       {FileFormatType::kParquet, GetNotImplementedFactory(FileFormatType::kParquet)},
       {FileFormatType::kOrc, GetNotImplementedFactory(FileFormatType::kOrc)},
       {FileFormatType::kPuffin, GetNotImplementedFactory(FileFormatType::kPuffin)},
-      {FileFormatType::kMetadata, GetNotImplementedFactory(FileFormatType::kMetadata)},
   };
   return factories.at(format_type);
 }

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,6 @@ enum class ICEBERG_EXPORT FileFormatType {`
`34`	`34`	`kAvro,`
`35`	`35`	`kOrc,`
`36`	`36`	`kPuffin,`
`37`		`- kMetadata,`
`38`	`37`	`};`
`39`	`38`
`40`	`39`	`/// \brief Convert a FileFormatType to a string`
`@@ -48,8 +47,6 @@ ICEBERG_EXPORT inline std::string_view ToString(FileFormatType format_type) {`
`48`	`47`	`return "orc";`
`49`	`48`	`case FileFormatType::kPuffin:`
`50`	`49`	`return "puffin";`
`51`		`- case FileFormatType::kMetadata:`
`52`		`- return "metadata";`
`53`	`50`	`}`
`54`	`51`	`}`
`55`	`52`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,6 @@ ReaderFactory& ReaderFactoryRegistry::GetFactory(FileFormatType format_type) {`
`42`	`42`	`{FileFormatType::kParquet, GetNotImplementedFactory(FileFormatType::kParquet)},`
`43`	`43`	`{FileFormatType::kOrc, GetNotImplementedFactory(FileFormatType::kOrc)},`
`44`	`44`	`{FileFormatType::kPuffin, GetNotImplementedFactory(FileFormatType::kPuffin)},`
`45`		`- {FileFormatType::kMetadata, GetNotImplementedFactory(FileFormatType::kMetadata)},`
`46`	`45`	`};`
`47`	`46`	`return factories.at(format_type);`
`48`	`47`	`}`