fix some comments

Author: gty404

src/iceberg/result.h

@@ -31,8 +31,8 @@ namespace iceberg {
 enum class ErrorKind {
   kAlreadyExists,
   kCommitStateUnknown,
-  kDataInvalid,
   kInvalidArgument,
+  kInvalidData,
   kInvalidExpression,
   kInvalidSchema,
   kIOError,
@@ -66,15 +66,15 @@ using Status = Result<void>;
 /// \brief Macro to define error creation functions
 #define DEFINE_ERROR_FUNCTION(name)                                           \
   template <typename... Args>                                                 \
-  inline auto name(const std::format_string<Args...> fmt, Args&&... args)     \
-      -> unexpected<Error> {                                                  \
+  inline auto name(const std::format_string<Args...> fmt,                     \
+                   Args&&... args) -> unexpected<Error> {                     \
     return unexpected<Error>(                                                 \
         {ErrorKind::k##name, std::format(fmt, std::forward<Args>(args)...)}); \
   }
 
 DEFINE_ERROR_FUNCTION(AlreadyExists)
 DEFINE_ERROR_FUNCTION(CommitStateUnknown)
-DEFINE_ERROR_FUNCTION(DataInvalid)
+DEFINE_ERROR_FUNCTION(InvalidData)
 DEFINE_ERROR_FUNCTION(InvalidArgument)
 DEFINE_ERROR_FUNCTION(InvalidExpression)
 DEFINE_ERROR_FUNCTION(InvalidSchema)

src/iceberg/table_scan.cc

@@ -60,7 +60,7 @@ Result<std::unique_ptr<TableScan>> TableScanBuilder::Build() {
       if (auto it = schemas.find(*snapshot->schema_id); it != schemas.end()) {
         return it->second;
       }
-      return DataInvalid("Schema {} in snapshot {} is not found", *snapshot->schema_id,
+      return InvalidData("Schema {} in snapshot {} is not found", *snapshot->schema_id,
                          snapshot->snapshot_id);
     }
     return table_.schema();