timing: Use analysis id as parent ID for latency data providers

This can be used for grouping on the front-end implementations. Only apply that for configured latency analysis where ITmfConfiguration is not null. [Updated] Latency Analysis DataProviderDescriptors with parentId Signed-off-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
eclipse-tracecompass · Oct 24, 2024 · 751b02c · 751b02c
1 parent cbb3866
commit 751b02c
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 0 deletions.
diff --git a/...ecompass/internal/analysis/profiling/core/instrumented/FlameChartDataProviderFactory.java b/...ecompass/internal/analysis/profiling/core/instrumented/FlameChartDataProviderFactory.java
@@ -99,6 +99,7 @@ public Collection<IDataProviderDescriptor> getDescriptors(ITmfTrace trace) {
             if (!existingModules.contains(analysis.getId())) {
                 DataProviderDescriptor.Builder builder = new DataProviderDescriptor.Builder();
                 builder.setId(FlameChartDataProvider.ID + DataProviderConstants.ID_SEPARATOR + analysis.getId())
+                        .setParentId(analysis.getConfiguration() != null ? analysis.getId() : null)
                         .setName(Objects.requireNonNull(analysis.getName() + " - " +Messages.FlameChartDataProvider_Title)) //$NON-NLS-1$
                         .setDescription(Objects.requireNonNull(NLS.bind(Messages.FlameChartDataProvider_Description, analysis.getHelpText())))
                         .setProviderType(ProviderType.TIME_GRAPH)

diff --git a/...alysis/timing/core/segmentstore/SegmentNameSegmentStoreStatisticsDataProviderFactory.java b/...alysis/timing/core/segmentstore/SegmentNameSegmentStoreStatisticsDataProviderFactory.java
@@ -53,6 +53,7 @@ protected void setStatisticsAnalysisModuleName(AbstractSegmentStatisticsAnalysis
     protected @Nullable IDataProviderDescriptor getDataProviderDescriptor(IAnalysisModule analysis) {
         DataProviderDescriptor.Builder builder = new DataProviderDescriptor.Builder();
         builder.setId(SegmentStoreStatisticsDataProvider.ID + DataProviderConstants.ID_SEPARATOR + analysis.getId())
+                .setParentId(analysis.getConfiguration() != null ? analysis.getId() : null)
                 .setName(Objects.requireNonNull(NLS.bind(Messages.SegmentStoreStatisticsDataProvider_title, analysis.getName())))
                 .setDescription(Objects.requireNonNull(NLS.bind(Messages.SegmentStoreStatisticsDataProvider_description, analysis.getHelpText())))
                 .setProviderType(ProviderType.DATA_TREE)

diff --git a/...ss/internal/analysis/timing/core/segmentstore/SegmentStoreScatterDataProviderFactory.java b/...ss/internal/analysis/timing/core/segmentstore/SegmentStoreScatterDataProviderFactory.java
@@ -73,6 +73,7 @@ public Collection<IDataProviderDescriptor> getDescriptors(ITmfTrace trace) {
             if (!existingModules.contains(analysis.getId())) {
                 DataProviderDescriptor.Builder builder = new DataProviderDescriptor.Builder();
                 builder.setId(SegmentStoreScatterDataProvider.ID + DataProviderConstants.ID_SEPARATOR + analysis.getId())
+                    .setParentId(analysis.getConfiguration() != null ? analysis.getId() : null)
                     .setName(Objects.requireNonNull(NLS.bind(Messages.SegmentStoreScatterGraphDataProvider_title, analysis.getName())))
                     .setDescription(Objects.requireNonNull(NLS.bind(Messages.SegmentStoreScatterGraphDataProvider_description, analysis.getHelpText())))
                     .setProviderType(ProviderType.TREE_TIME_XY)

diff --git a/...pass/internal/analysis/timing/core/segmentstore/SegmentStoreTableDataProviderFactory.java b/...pass/internal/analysis/timing/core/segmentstore/SegmentStoreTableDataProviderFactory.java
@@ -77,6 +77,7 @@ public Collection<IDataProviderDescriptor> getDescriptors(ITmfTrace trace) {
             if (!existingModules.contains(analysis.getId())) {
                 DataProviderDescriptor.Builder builder = new DataProviderDescriptor.Builder();
                 builder.setId(SegmentStoreTableDataProvider.ID + DataProviderConstants.ID_SEPARATOR + analysis.getId())
+                        .setParentId(analysis.getConfiguration() != null ? analysis.getId() : null)
                         .setName(Objects.requireNonNull(NLS.bind(Messages.SegmentStoreTableDataProvider_title, analysis.getName())))
                         .setDescription(Objects.requireNonNull(NLS.bind(Messages.SegmentStoreTableDataProvider_description, analysis.getHelpText())))
                         .setProviderType(ProviderType.TABLE)