Skip to content

Commit 45805c8

Browse files
committed
fix
1 parent 63f9aa3 commit 45805c8

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

src/query/service/src/interpreters/interpreter_explain.rs

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -197,10 +197,7 @@ impl Interpreter for ExplainInterpreter {
197197
schema.clone(),
198198
metadata.clone(),
199199
)?;
200-
let mut plan = interpreter.build_physical_plan(&mutation, None).await?;
201-
if let PhysicalPlan::CommitSink(commit) = &plan {
202-
plan = commit.input.as_ref().clone();
203-
}
200+
let plan = interpreter.build_physical_plan(&mutation, None).await?;
204201
self.explain_analyze(plan, metadata, true).await?
205202
}
206203
_ => Err(ErrorCode::Unimplemented(
@@ -442,13 +439,18 @@ impl ExplainInterpreter {
442439
metadata: &MetadataRef,
443440
ignore_result: bool,
444441
) -> Result<Vec<DataBlock>> {
445-
let build_res = build_query_pipeline(&self.ctx, &[], &plan, ignore_result).await?;
442+
let plan = if let PhysicalPlan::CommitSink(commit) = &plan {
443+
commit.input.as_ref()
444+
} else {
445+
&plan
446+
};
447+
let build_res = build_query_pipeline(&self.ctx, &[], plan, ignore_result).await?;
446448

447449
// Drain the data
448450
let query_profiles = self.execute_and_get_profiles(build_res)?;
449451

450452
let result = if self.partial {
451-
format_partial_tree(&plan, metadata, &query_profiles)?.format_pretty()?
453+
format_partial_tree(plan, metadata, &query_profiles)?.format_pretty()?
452454
} else {
453455
plan.format(metadata.clone(), query_profiles.clone())?
454456
.format_pretty()?

src/query/storages/fuse/src/operations/mutation_source.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,6 @@ impl FuseTable {
281281
let mut num_whole_block_mutation = whole_block_deletions.len();
282282
let segment_num = pruner.deleted_segments.len();
283283
// now try to add deleted_segment
284-
// todo check
285284
for deleted_segment in pruner.deleted_segments {
286285
part_num += deleted_segment.summary.block_count as usize;
287286
num_whole_block_mutation += deleted_segment.summary.block_count as usize;

0 commit comments

Comments
 (0)