Skip to content

Commit

Permalink
[GLUTEN-7202][CH] Fix: local executor cannot dump pipeline stats apac…
Browse files Browse the repository at this point in the history
…he#7204

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)

Fixes: apache#7202

How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)

unit tests

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
  • Loading branch information
lgbo-ustc authored and shamirchen committed Oct 14, 2024
1 parent 956d25e commit 86fc749
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion cpp-ch/local-engine/Parser/LocalExecutor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,16 @@ LocalExecutor::LocalExecutor(QueryPlanPtr query_plan, QueryPipelineBuilderPtr pi
thread_local LocalExecutor * LocalExecutor::current_executor = nullptr;
std::string LocalExecutor::dumpPipeline() const
{
const auto & processors = query_pipeline.getProcessors();
const DB::Processors * processors_ref = nullptr;
if (push_executor)
{
processors_ref = &(push_executor->getProcessors());
}
else
{
processors_ref = &(query_pipeline.getProcessors());
}
const auto & processors = *processors_ref;
for (auto & processor : processors)
{
WriteBufferFromOwnString buffer;
Expand Down

0 comments on commit 86fc749

Please sign in to comment.