eclipse-cdt-cloud / eclipse-cdt-cloud/trace-server-protocol
Consolidate input query parameters across views for time ranges
- Vorherrschende Sprache
- Python
- Sterne
- 26
- Forks
- 21
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
### Bug Description:
Consolidate input query parameters across views for time ranges.
Any time-based views uses the `requested_timerange` as input parameter to query the data for a give time range. Other views which don't have the trace time as x-axis don't always use `requested_timerange` for that.
For example, the GANTT_CHART data provider uses the requested_timerange to specify the start duration and end duration if the x-axis is duration. If the x-axis would be other data, e.g. bytes this input parameter would make even less sense. For the GANTT_CHART another query parameter exists in the Trace Compass to specify the time range to apply (see [here](https://github.com/eclipse-tracecompass/org.eclipse.tracecompass/blob/d487cf2e11795a2680d3333677f443e98724bcb9/analysis/org.eclipse.tracecompass.analysis.profiling.core/src/org/eclipse/tracecompass/internal/analysis/profiling/core/flamegraph/FlameGraphDataProvider.java#L506).
Another example are the Latency views, where it's possible to add the query parameter `requested_times` to specify start time (value at index 0) and end time (value at index 1). Additionally it is required to add the filter flag `isFiltered` with value true in the query parameters. Adding this will add total statistics and the time range based statistics in one query.
A third example is the generic xy chart implementation under development (see [here](https://github.com/eclipse-tracecompass/org.eclipse.tracecompass/pull/291#pullrequestreview-3055341970)) where the parameter `requested_timerange` is used for querying the data provider for specific time range.
This ticket is to align the input parameters and behaviour across data providers.
### Additional Information
N/A
Beitragsleitfaden
Rechercherichtung
Beginnen Sie damit, die Behandlung von requested_timerange und requested_times im GANTT_CHART-Provider, in FlameGraphDataProvider.java, in den Latency-Ansichten und in der in PR #291 referenzierten Arbeit an generischen XY-Charts zu vergleichen. Für das Issue müssen ein abgestimmter Vertrag für Eingabeparameter und ein konsistentes Verhalten über alle Datenprovider hinweg festgelegt werden; der Abschluss sollte für jede genannte Ansicht anhand dieses Vertrags überprüft werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100