Skip to content

Commit

Permalink
update review
Browse files Browse the repository at this point in the history
  • Loading branch information
zhangstar333 committed Sep 19, 2023
1 parent b0b4f19 commit 7324bef
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -97,14 +97,8 @@ OperatorPtr DistinctStreamingAggSinkOperatorBuilder::build_operator() {

DistinctStreamingAggSinkLocalState::DistinctStreamingAggSinkLocalState(
DataSinkOperatorXBase* parent, RuntimeState* state)
: AggSinkLocalState<AggDependency, DistinctStreamingAggSinkLocalState>(parent, state) {
dummy_mapped_data = std::make_shared<char>('A');
}

Status DistinctStreamingAggSinkLocalState::init(RuntimeState* state, LocalSinkStateInfo& info) {
RETURN_IF_ERROR(Base::init(state, info));
return Status::OK();
}
: AggSinkLocalState<AggDependency, DistinctStreamingAggSinkLocalState>(parent, state),
dummy_mapped_data(std::make_shared<char>('A')) {}

Status DistinctStreamingAggSinkLocalState::_distinct_pre_agg_with_serialized_key(
doris::vectorized::Block* in_block, doris::vectorized::Block* out_block) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ class DistinctStreamingAggSinkLocalState final
ENABLE_FACTORY_CREATOR(DistinctStreamingAggSinkLocalState);
DistinctStreamingAggSinkLocalState(DataSinkOperatorXBase* parent, RuntimeState* state);

Status init(RuntimeState* state, LocalSinkStateInfo& info) override;
Status close(RuntimeState* state) override;
Status _distinct_pre_agg_with_serialized_key(vectorized::Block* in_block,
vectorized::Block* out_block);
Expand Down Expand Up @@ -114,4 +113,4 @@ class DistinctStreamingAggSinkOperatorX final
};

} // namespace pipeline
} // namespace doris
} // namespace doris
Original file line number Diff line number Diff line change
Expand Up @@ -79,4 +79,4 @@ class DistinctStreamingAggSourceOperatorX final : public AggSourceOperatorX {
};

} // namespace pipeline
} // namespace doris
} // namespace doris

0 comments on commit 7324bef

Please sign in to comment.