ClickHouse/src/DataStreams/BlockIO.cpp

52 lines
1.3 KiB
C++
Raw Normal View History

2020-02-27 15:40:11 +00:00
#include <DataStreams/BlockIO.h>
#include <Interpreters/ProcessList.h>
namespace DB
{
2020-02-27 15:40:11 +00:00
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
2020-08-06 12:24:05 +00:00
pipeline.reset();
2020-02-27 15:40:11 +00:00
process_list_entry.reset();
/// TODO Do we need also reset callbacks? In which order?
}
BlockIO & BlockIO::operator= (BlockIO && rhs)
{
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 = std::move(rhs.null_format);
return *this;
}
BlockIO::~BlockIO()
{
reset();
}
}