Add flag to control reordering metrics for display
Open
- 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
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