feat: add DataFile aggregate evaluation

Author: SuKi2cn

src/iceberg/expression/expression.h

@@ -77,7 +77,7 @@ class ICEBERG_EXPORT Expression : public util::Formattable {
   /// \brief Returns whether this expression will accept the same values as another.
   /// \param other another expression
   /// \return true if the expressions are equivalent
-  virtual bool Equals([[maybe_unused]] const Expression& other) const {
+  virtual bool Equals(const Expression& other) const {
     // only bound predicates can be equivalent
     return false;
   }

src/iceberg/expression/expression_visitor.h

@@ -117,13 +117,13 @@ class ICEBERG_EXPORT BoundVisitor : public ExpressionVisitor<R> {
 
   /// \brief Visit an IS_NAN bound expression.
   /// \param expr The bound expression being tested
-  virtual Result<R> IsNaN([[maybe_unused]] const std::shared_ptr<Bound>& expr) {
+  virtual Result<R> IsNaN(const std::shared_ptr<Bound>& expr) {
     return NotSupported("IsNaN operation is not supported by this visitor");
   }
 
   /// \brief Visit a NOT_NAN bound expression.
   /// \param expr The bound expression being tested
-  virtual Result<R> NotNaN([[maybe_unused]] const std::shared_ptr<Bound>& expr) {
+  virtual Result<R> NotNaN(const std::shared_ptr<Bound>& expr) {
     return NotSupported("NotNaN operation is not supported by this visitor");
   }
 

src/iceberg/schema.h

@@ -51,8 +51,6 @@ class ICEBERG_EXPORT Schema : public StructType {
   explicit Schema(std::vector<SchemaField> fields,
                   std::optional<int32_t> schema_id = std::nullopt);
 
-  using StructType::Equals;
-
   /// \brief Get the schema ID.
   ///
   /// A schema is identified by a unique ID for the purposes of schema

src/iceberg/test/aggregate_test.cc

@@ -23,7 +23,7 @@
 
 #include "iceberg/expression/binder.h"
 #include "iceberg/expression/expressions.h"
-#include "iceberg/manifest_entry.h"
+#include "iceberg/manifest/manifest_entry.h"
 #include "iceberg/row/struct_like.h"
 #include "iceberg/schema.h"
 #include "iceberg/test/matchers.h"