Skip to content

Conversation

@bhufmann
Copy link
Contributor

This cherry-pick PR includes the commits of following PR:

Signed-off-by: Bernd Hufmann [email protected]

As the DPDK analysis uses the DpdkTrace format, relocate all the
analysis files to the 'tracetypes' directory for better organization.
Using bookmarks, user can save custom data points where each corresponds
to a specific time range.
Each bookmark contains "name", "start", "end", and an optional
"payload".

The following endpoints are introduced:
- [GET] Get all the bookmarks of an experiment
- [GET] Get a specific bookmark of an experiment
- [POST] Create a new bookmark for an experiment
- [PUT] Update an old bookmark of an experiment
- [DELETE] Delete a bookmark of an experiment

[Added] Corresponding endpoints for bookmarks in trace server

Signed-off-by: Kaveh Shahedi <[email protected]>
Required unit tests have been implemented for BookmarkServiceManager class. In the tests,
various aspects of bookmarking functionality (e.g., creating, updating, deleting, etc.)
are checked.

[Added] Required unit tests for bookmarking functionality

Signed-off-by: Kaveh Shahedi <[email protected]>
The new changes aim to re-structure the algorithm of storing bookmarks.
Previously, we stored each bookmark (of an experiment) under a specific
directory inside the .webapp directory of Trace Compass. Right now, we
are using Eclipse's markers system to store and fetch the benchmarks.

[Changed] Bookmark storing/fetching system is changed to IMarker

Signed-off-by: Kaveh Shahedi <[email protected]>
The version is now updated to 0.3.0 from 0.2.0. Most of the changes
are related to managing experiments' bookmarks. Also, the bookmarks
tag (BKM) is added to TraceServerOpenApiResource class.

[Changed] Trace server version is changed to 0.3.0

Signed-off-by: Kaveh Shahedi <[email protected]>
@bhufmann bhufmann merged commit 273bf46 into eclipse-tracecompass-incubator:2021-06 Nov 27, 2024
2 checks passed
@bhufmann bhufmann deleted the cherry-pick branch November 27, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants