Skip to content

Commit 561e31d

Browse files
author
Xinpeng Wei
committed
fmt: fix clippy warning
Signed-off-by: Xinpeng Wei <xinpeng@singularity-data.com>
1 parent da0af9d commit 561e31d

File tree

1 file changed

+20
-20
lines changed

1 file changed

+20
-20
lines changed

‎src/executor/mod.rs

+20-20
Original file line numberDiff line numberDiff line change
@@ -209,12 +209,12 @@ impl ExecutorExt for BoxedExecutor {
209209

210210
impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
211211
fn visit_dummy(&mut self, _plan: &Dummy, root: &Span) -> Option<BoxedExecutor> {
212-
let _child_span = Span::enter_with_parent("DummyScanExecutor", &root);
212+
let _child_span = Span::enter_with_parent("DummyScanExecutor", root);
213213
Some(DummyScanExecutor.execute())
214214
}
215215

216216
fn visit_internal(&mut self, plan: &Internal, root: &Span) -> Option<BoxedExecutor> {
217-
let _child_span = Span::enter_with_parent("InternalTableExecutor", &root);
217+
let _child_span = Span::enter_with_parent("InternalTableExecutor", root);
218218
Some(
219219
InternalTableExecutor {
220220
table_name: plan.table_name(),
@@ -228,7 +228,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
228228
plan: &PhysicalCreateTable,
229229
root: &Span,
230230
) -> Option<BoxedExecutor> {
231-
let _child_span = Span::enter_with_parent("CreateTableExecutor", &root);
231+
let _child_span = Span::enter_with_parent("CreateTableExecutor", root);
232232
Some(match &self.storage {
233233
StorageImpl::InMemoryStorage(storage) => CreateTableExecutor {
234234
plan: plan.clone(),
@@ -244,7 +244,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
244244
}
245245

246246
fn visit_physical_drop(&mut self, plan: &PhysicalDrop, root: &Span) -> Option<BoxedExecutor> {
247-
let _child_span = Span::enter_with_parent("DropExecutor", &root);
247+
let _child_span = Span::enter_with_parent("DropExecutor", root);
248248
Some(match &self.storage {
249249
StorageImpl::InMemoryStorage(storage) => DropExecutor {
250250
plan: plan.clone(),
@@ -264,7 +264,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
264264
plan: &PhysicalInsert,
265265
root: &Span,
266266
) -> Option<BoxedExecutor> {
267-
let _child_span = Span::enter_with_parent("InsertExecutor", &root);
267+
let _child_span = Span::enter_with_parent("InsertExecutor", root);
268268
Some(match &self.storage {
269269
StorageImpl::InMemoryStorage(storage) => InsertExecutor {
270270
context: self.context.clone(),
@@ -292,7 +292,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
292292
plan: &PhysicalNestedLoopJoin,
293293
root: &Span,
294294
) -> Option<BoxedExecutor> {
295-
let _child_span = Span::enter_with_parent("NestedLoopJoinExecutor", &root);
295+
let _child_span = Span::enter_with_parent("NestedLoopJoinExecutor", root);
296296
let left_child = self.visit(plan.left(), &_child_span).unwrap();
297297
let right_child = self.visit(plan.right(), &_child_span).unwrap();
298298
Some(
@@ -313,7 +313,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
313313
plan: &PhysicalTableScan,
314314
root: &Span,
315315
) -> Option<BoxedExecutor> {
316-
let _child_span = Span::enter_with_parent("TableScanExecutor", &root);
316+
let _child_span = Span::enter_with_parent("TableScanExecutor", root);
317317
Some(match &self.storage {
318318
StorageImpl::InMemoryStorage(storage) => TableScanExecutor {
319319
context: self.context.clone(),
@@ -339,7 +339,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
339339
plan: &PhysicalProjection,
340340
root: &Span,
341341
) -> Option<BoxedExecutor> {
342-
let _child_span = Span::enter_with_parent("ProjectionExecutor", &root);
342+
let _child_span = Span::enter_with_parent("ProjectionExecutor", root);
343343
Some(
344344
ProjectionExecutor {
345345
project_expressions: plan.logical().project_expressions().to_vec(),
@@ -354,7 +354,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
354354
plan: &PhysicalFilter,
355355
root: &Span,
356356
) -> Option<BoxedExecutor> {
357-
let _child_span = Span::enter_with_parent("FilterExecutor", &root);
357+
let _child_span = Span::enter_with_parent("FilterExecutor", root);
358358
Some(
359359
FilterExecutor {
360360
expr: plan.logical().expr().clone(),
@@ -365,7 +365,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
365365
}
366366

367367
fn visit_physical_order(&mut self, plan: &PhysicalOrder, root: &Span) -> Option<BoxedExecutor> {
368-
let _child_span = Span::enter_with_parent("OrderExecutor", &root);
368+
let _child_span = Span::enter_with_parent("OrderExecutor", root);
369369
Some(
370370
OrderExecutor {
371371
comparators: plan.logical().comparators().to_vec(),
@@ -376,7 +376,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
376376
}
377377

378378
fn visit_physical_limit(&mut self, plan: &PhysicalLimit, root: &Span) -> Option<BoxedExecutor> {
379-
let _child_span = Span::enter_with_parent("a child span", &root);
379+
let _child_span = Span::enter_with_parent("a child span", root);
380380
Some(
381381
LimitExecutor {
382382
child: self.visit(plan.child(), &_child_span).unwrap(),
@@ -388,7 +388,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
388388
}
389389

390390
fn visit_physical_top_n(&mut self, plan: &PhysicalTopN, root: &Span) -> Option<BoxedExecutor> {
391-
let _child_span = Span::enter_with_parent("TopNExecutor", &root);
391+
let _child_span = Span::enter_with_parent("TopNExecutor", root);
392392
Some(
393393
TopNExecutor {
394394
child: self.visit(plan.child(), &_child_span).unwrap(),
@@ -405,7 +405,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
405405
plan: &PhysicalExplain,
406406
root: &Span,
407407
) -> Option<BoxedExecutor> {
408-
let _child_span = Span::enter_with_parent("ExplainExecutor", &root);
408+
let _child_span = Span::enter_with_parent("ExplainExecutor", root);
409409
Some(ExplainExecutor { plan: plan.clone() }.execute())
410410
}
411411

@@ -414,7 +414,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
414414
plan: &PhysicalHashAgg,
415415
root: &Span,
416416
) -> Option<BoxedExecutor> {
417-
let _child_span = Span::enter_with_parent("HashAggExecutor", &root);
417+
let _child_span = Span::enter_with_parent("HashAggExecutor", root);
418418
Some(
419419
HashAggExecutor {
420420
agg_calls: plan.logical().agg_calls().to_vec(),
@@ -430,7 +430,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
430430
plan: &PhysicalHashJoin,
431431
root: &Span,
432432
) -> Option<BoxedExecutor> {
433-
let _child_span = Span::enter_with_parent("HashJoinExecutor", &root);
433+
let _child_span = Span::enter_with_parent("HashJoinExecutor", root);
434434
let left_child = self.visit(plan.left(), &_child_span).unwrap();
435435
let right_child = self.visit(plan.right(), &_child_span).unwrap();
436436

@@ -462,7 +462,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
462462
plan: &PhysicalSimpleAgg,
463463
root: &Span,
464464
) -> Option<BoxedExecutor> {
465-
let _child_span = Span::enter_with_parent("SimpleAggExecutor", &root);
465+
let _child_span = Span::enter_with_parent("SimpleAggExecutor", root);
466466
Some(
467467
SimpleAggExecutor {
468468
agg_calls: plan.agg_calls().to_vec(),
@@ -477,7 +477,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
477477
plan: &PhysicalDelete,
478478
root: &Span,
479479
) -> Option<BoxedExecutor> {
480-
let _child_span = Span::enter_with_parent("DeleteExecutor", &root);
480+
let _child_span = Span::enter_with_parent("DeleteExecutor", root);
481481
let child = self.visit(plan.child(), &_child_span).unwrap();
482482
Some(match &self.storage {
483483
StorageImpl::InMemoryStorage(storage) => DeleteExecutor {
@@ -504,7 +504,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
504504
plan: &PhysicalValues,
505505
root: &Span,
506506
) -> Option<BoxedExecutor> {
507-
let _child_span = Span::enter_with_parent("ValuesExecutor", &root);
507+
let _child_span = Span::enter_with_parent("ValuesExecutor", root);
508508
Some(
509509
ValuesExecutor {
510510
column_types: plan.logical().column_types().to_vec(),
@@ -519,7 +519,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
519519
plan: &PhysicalCopyFromFile,
520520
root: &Span,
521521
) -> Option<BoxedExecutor> {
522-
let _child_span = Span::enter_with_parent("CopyFromFileExecutor", &root);
522+
let _child_span = Span::enter_with_parent("CopyFromFileExecutor", root);
523523
Some(
524524
CopyFromFileExecutor {
525525
context: self.context.clone(),
@@ -535,7 +535,7 @@ impl PlanVisitor<BoxedExecutor> for ExecutorBuilder {
535535
plan: &PhysicalCopyToFile,
536536
root: &Span,
537537
) -> Option<BoxedExecutor> {
538-
let _child_span = Span::enter_with_parent("CopyToFileExecutor", &root);
538+
let _child_span = Span::enter_with_parent("CopyToFileExecutor", root);
539539
Some(
540540
CopyToFileExecutor {
541541
context: self.context.clone(),

0 commit comments

Comments
 (0)