thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-22 07:31:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

sumMapFiltered: use CRTP instead of virtual for keepKey

Browse Source
This commit is contained in:
Léo Ercolanelli 2019-01-24 09:02:17 +01:00
parent efb1cb0528
commit 22f7bd03b9
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
dbms/src/AggregateFunctions/AggregateFunctionSumMap.h
View File

@ -259,7 +259,7 @@ public:
const char * getHeaderFilePath() const override { return __FILE__; }
virtual bool keepKey(const T & key) const = 0;
bool keepKey(const T & key) const { return static_cast<const Derived &>(*this).keepKey(key); }
};
template <typename T>
@ -272,7 +272,7 @@ public:
String getName() const override { return "sumMap"; }
bool keepKey(const T &) const override { return true; }
bool keepKey(const T &) const { return true; }
};
template <typename T>
@ -295,7 +295,7 @@ public:
String getName() const override { return "sumMapFiltered"; }
bool keepKey(const T & key) const override
bool keepKey(const T & key) const
{
return std::binary_search(begin(keys_to_keep), end(keys_to_keep), key);
}