Skip to content

Commit

Permalink
Update cpp/src/aggregation/result_cache.cpp
Browse files Browse the repository at this point in the history
Co-authored-by: Bradley Dice <[email protected]>
  • Loading branch information
karthikeyann and bdice authored Oct 26, 2021
1 parent 9e1a41a commit 96fef91
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions cpp/src/aggregation/result_cache.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,6 @@ void result_cache::add_result(column_view const& input,
aggregation const& agg,
std::unique_ptr<column>&& col)
{
// skip insert if the result is already in the cache
if (has_result(input, agg)) return;
// We can't guarantee that agg will outlive the cache, so we need to take ownership of a copy.
// To allow lookup by reference, make the key a reference and keep the owner in the value pair.
auto owned_agg = agg.clone();
Expand Down

0 comments on commit 96fef91

Please sign in to comment.