revert new error code

Author: xiao.dong

src/iceberg/expression/expression_visitor.h

@@ -263,7 +263,7 @@ class ICEBERG_EXPORT BoundVisitor : public ExpressionVisitor<R> {
   /// Bound visitors do not support unbound predicates.
   ///
   /// \param pred The unbound predicate
-  Result<R> Predicate(const std::shared_ptr<UnboundPredicate>& pred) override {
+  Result<R> Predicate(const std::shared_ptr<UnboundPredicate>& pred) final {
     ICEBERG_DCHECK(pred != nullptr, "UnboundPredicate cannot be null");
     return NotSupported("Not a bound predicate: {}", pred->ToString());
   }

src/iceberg/expression/inclusive_metrics_evaluator.cc

@@ -434,7 +434,7 @@ class InclusiveMetricsVisitor : public BoundVisitor<bool> {
     int id = ref.field().field_id();
     auto type = ref.type();
     if (!type->is_primitive()) {
-      return InvalidStats("Lower bound of non-primitive type is not supported.");
+      return Invalid("Lower bound of non-primitive type is not supported.");
     }
     auto primitive_type = std::dynamic_pointer_cast<PrimitiveType>(type);
     if (!data_file_.lower_bounds.empty() && data_file_.lower_bounds.contains(id)) {
@@ -451,7 +451,7 @@ class InclusiveMetricsVisitor : public BoundVisitor<bool> {
     int id = ref.field().field_id();
     auto type = ref.type();
     if (!type->is_primitive()) {
-      return InvalidStats("Upper bound of non-primitive type is not supported.");
+      return Invalid("Upper bound of non-primitive type is not supported.");
     }
     auto primitive_type = std::dynamic_pointer_cast<PrimitiveType>(type);
     if (!data_file_.upper_bounds.empty() && data_file_.upper_bounds.contains(id)) {

src/iceberg/result.h

@@ -43,7 +43,6 @@ enum class ErrorKind {
   kInvalidManifest,
   kInvalidManifestList,
   kInvalidSchema,
-  kInvalidStats,
   kIOError,
   kJsonParseError,
   kNamespaceNotEmpty,
@@ -105,7 +104,6 @@ DEFINE_ERROR_FUNCTION(InvalidExpression)
 DEFINE_ERROR_FUNCTION(InvalidManifest)
 DEFINE_ERROR_FUNCTION(InvalidManifestList)
 DEFINE_ERROR_FUNCTION(InvalidSchema)
-DEFINE_ERROR_FUNCTION(InvalidStats)
 DEFINE_ERROR_FUNCTION(IOError)
 DEFINE_ERROR_FUNCTION(JsonParseError)
 DEFINE_ERROR_FUNCTION(NamespaceNotEmpty)