apache / apache/datafusion

Add flag to control reordering metrics for display

Open
#15,992 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Rust
Stars
9.3k
Forks
2.4k
Avg merge
3d 7h
Merged PRs (30d)
344

Description

`DisplayableExecutionPlan::indent` always reorder metrics. However, ones might expect the order of the metrics conform to the registration order.

I propose to add a flag to control the reordering behaviour. For example:

```
DisplayableExecutionPlan::new(&plan).set_reorder(false).indent(false)
```

Contributor guide

Open the contributing guide

Research direction

Start by locating DisplayableExecutionPlan::new and indent, then inspect nearby tests or callers to understand the current metric reordering. Add coverage showing that the default behavior remains unchanged and that disabling reordering preserves registration order.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
observability
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.