adoptium / adoptium/aqa-test-tools

Optimize Parsing & Storage of Raw Values

Open
#193 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Jupyter Notebook
Stars
33
Forks
97
Avg merge
7h 9m
Merged PRs (30d)
5

Description

### Problem Description

In order to extract Perf results for builds that have multiple tests in them (https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/136), we decided to store raw values in the aggregate info data structure.

The **design** is discussed: https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/136#issuecomment-582062349

**Pros of current design:**

- Makes parser simpler
- Needs to done only once when the build finishes
- Different front end tools doesn't require special parsing code.
- Avoids somewhat duplication of code in the front end since front end tools would require similar logic as the one that's already there in the back end.

**Cons of current design:**

- Duplicates raw values by collecting it from different places/builds and putting them in an array

We could re-visit this design to see whether we can make this more efficient to possibly avoid duplicating raw values.

**Related PR:** https://github.com/AdoptOpenJDK/openjdk-test-tools/pull/179

Contributor guide

Open the contributing guide

Research direction

Start by reading the design discussion in issue #136 and the related PR #179, which are the only implementation references provided. Determine whether the aggregate raw-value storage can be made more efficient without losing the stated support for builds with multiple tests; done means an agreed design and corresponding implementation that avoids unnecessary duplication.

Written by the indexing model from the issue text.

Assessment

Domain
data, performance
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.