thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-20 08:40:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Better function name and comment

Browse Source
This commit is contained in:
alesapin 2020-07-21 11:13:23 +03:00
parent 82c5ff6e9b
commit 1d8cab3379
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Dictionaries/CacheDictionary.cpp
View File

@ -865,7 +865,7 @@ void CacheDictionary::update(BunchUpdateUnit & bunch_update_unit) const
{
try
{
auto current_source_ptr = getDictionarySourceOrUpdate();
auto current_source_ptr = getSourceAndUpdateIfNeeded();
Stopwatch watch;

6
src/Dictionaries/CacheDictionary.h
View File

@ -92,7 +92,7 @@ public:
database,
name,
dict_struct,
getDictionarySourceOrUpdate()->clone(),
getSourceAndUpdateIfNeeded()->clone(),
dict_lifetime,
strict_max_lifetime_seconds,
size,
@ -295,7 +295,7 @@ private:
/// MultiVersion is not used here because it works with constant pointers.
/// For some reason almost all methods in IDictionarySource interface are
/// not constant.
SharedDictionarySourcePtr getDictionarySourceOrUpdate() const
SharedDictionarySourcePtr getSourceAndUpdateIfNeeded() const
{
std::lock_guard lock(source_mutex);
if (error_count)
@ -326,7 +326,7 @@ private:
const std::string full_name;
const DictionaryStructure dict_struct;
/// Dictionary source should be used without mutex
/// Dictionary source should be used with mutex
mutable std::mutex source_mutex;
mutable SharedDictionarySourcePtr source_ptr;