Skip to content

Commit

Permalink
Aggregate name for null count.
Browse files Browse the repository at this point in the history
  • Loading branch information
KiterLuc committed Oct 23, 2023
1 parent 28b5ec1 commit e316393
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions tiledb/sm/query/readers/aggregators/count_aggregator.h
Original file line number Diff line number Diff line change
Expand Up @@ -101,11 +101,6 @@ class CountAggregatorBase : public OutputBufferValidator, public IAggregator {
std::string output_field_name,
std::unordered_map<std::string, QueryBuffer>& buffers) override;

/** Returns name of the aggregate. */
std::string aggregate_name() override {
return constants::aggregate_count_str;
}

/** Returns the TileDB datatype of the output field for the aggregate. */
Datatype output_datatype() override {
return Datatype::UINT64;
Expand Down Expand Up @@ -142,6 +137,11 @@ class CountAggregator : public CountAggregatorBase<NonNull> {
std::string field_name() override {
return constants::count_of_rows;
}

/** Returns name of the aggregate. */
std::string aggregate_name() override {
return constants::aggregate_count_str;
}
};

class NullCountAggregator : public CountAggregatorBase<Null>,
Expand All @@ -162,6 +162,11 @@ class NullCountAggregator : public CountAggregatorBase<Null>,
return field_info_.name_;
}

/** Returns name of the aggregate. */
std::string aggregate_name() override {
return constants::aggregate_null_count_str;
}

private:
/* ********************************* */
/* PRIVATE ATTRIBUTES */
Expand Down

0 comments on commit e316393

Please sign in to comment.