mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-12-16 19:32:07 +00:00
996d838ca3
Clang -Tidy Fixes
62 lines
1.6 KiB
C++
62 lines
1.6 KiB
C++
#include <QueryPipeline/BlockIO.h>
|
|
#include <Interpreters/ProcessList.h>
|
|
|
|
namespace DB
|
|
{
|
|
|
|
|
|
void BlockIO::reset()
|
|
{
|
|
/** process_list_entry should be destroyed after in, after out and after pipeline,
|
|
* since in, out and pipeline contain pointer to objects inside process_list_entry (query-level MemoryTracker for example),
|
|
* which could be used before destroying of in and out.
|
|
*
|
|
* However, QueryStatus inside process_list_entry holds shared pointers to streams for some reason.
|
|
* Streams must be destroyed before storage locks, storages and contexts inside pipeline,
|
|
* so releaseQueryStreams() is required.
|
|
*/
|
|
/// TODO simplify it all
|
|
|
|
pipeline.reset();
|
|
process_list_entry.reset();
|
|
|
|
/// TODO Do we need also reset callbacks? In which order?
|
|
}
|
|
|
|
BlockIO & BlockIO::operator= (BlockIO && rhs) noexcept
|
|
{
|
|
if (this == &rhs)
|
|
return *this;
|
|
|
|
/// Explicitly reset fields, so everything is destructed in right order
|
|
reset();
|
|
|
|
process_list_entry = std::move(rhs.process_list_entry);
|
|
pipeline = std::move(rhs.pipeline);
|
|
|
|
finish_callback = std::move(rhs.finish_callback);
|
|
exception_callback = std::move(rhs.exception_callback);
|
|
|
|
null_format = rhs.null_format;
|
|
|
|
return *this;
|
|
}
|
|
|
|
BlockIO::~BlockIO()
|
|
{
|
|
reset();
|
|
}
|
|
|
|
void BlockIO::setAllDataSent() const
|
|
{
|
|
/// The following queries does not have process_list_entry:
|
|
/// - internal
|
|
/// - SHOW PROCESSLIST
|
|
if (process_list_entry)
|
|
(*process_list_entry)->setAllDataSent();
|
|
}
|
|
|
|
|
|
}
|
|
|