refactor: mark aggregate parameters as maybe unused in expression visitor

Author: Joe-Abraham

src/iceberg/expression/expression_visitor.h

@@ -82,15 +82,17 @@ class ICEBERG_EXPORT ExpressionVisitor {
 
   /// \brief Visit a bound aggregate.
   /// \param aggregate The bound aggregate to visit.
-  virtual Result<R> Aggregate(const std::shared_ptr<BoundAggregate>& aggregate) {
+  virtual Result<R> Aggregate(
+      [[maybe_unused]] const std::shared_ptr<BoundAggregate>& aggregate) {
     ICEBERG_DCHECK(aggregate != nullptr, "Bound aggregate cannot be null");
     return NotSupported("Visitor {} does not support bound aggregate",
                         typeid(*this).name());
   }
 
   /// \brief Visit an unbound aggregate.
   /// \param aggregate The unbound aggregate to visit.
-  virtual Result<R> Aggregate(const std::shared_ptr<UnboundAggregate>& aggregate) {
+  virtual Result<R> Aggregate(
+      [[maybe_unused]] const std::shared_ptr<UnboundAggregate>& aggregate) {
     ICEBERG_DCHECK(aggregate != nullptr, "Unbound aggregate cannot be null");
     return NotSupported("Visitor {} does not support unbound aggregate",
                         typeid(*this).name());