…r iteration

- added measurement table migration script

Co-authored-by: Humphrey Burchell <[email protected]>
Co-authored-by: Stefan Marr <[email protected]>
Signed-off-by: Stefan Marr <[email protected]>

… of Measurement.values

- add a little more typing
- rename Measurement.value to values
- refactor data preparation to simply create value batches for insertion
- remove old test not needed anymore
- updated small-payload.json to new API format
- only consider non-null values for timeline computation

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

This is not a feature to be exposed, this is merely intended for testing, and converting the large-payload file.
It also helps to define more explicitly what the complete API definition is, because of missing values and criteria without data.

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

This is hopefully easier to use, and reports errors consistently.

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Some of these numbers are still odd, but oh well…

Signed-off-by: Stefan Marr <[email protected]>

… database id

Signed-off-by: Stefan Marr <[email protected]>

… benchmarks

It complains as follows, which we can just avoid:

file:///project/dist/tests/helpers.js:1
        import { expect } from '@jest/globals';
                 ^^^^^^
        SyntaxError: Named export 'expect' not found. The requested module '@jest/globals' is a CommonJS module, which may not support all module.exports as named exports.
        CommonJS modules can always be imported via the default export, for example using:
Signed-off-by: Stefan Marr <[email protected]>

- load old format, do sizing as before, and then convert

Signed-off-by: Stefan Marr <[email protected]>

This is mostly to have them covered…

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

… optimization

- rename methods for benchmarking (also used for convenience in test)
- avoid creating empty jobs

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

…ion ids being used.

We didn’t increment the criterionId when no measurements were available, which lead to a wrong mapping of the ids used in the input data to ids used in the database.

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

- account for new data format with ValuesPossiblyMissing

Signed-off-by: Stefan Marr <[email protected]>

The functions are in the next commit.

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

…the ReBench data file format

- merge redundant type declaration
- filter out NULL value in the database

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

Signed-off-by: Stefan Marr <[email protected]>

The corresponding PR for the new API support is now merged. And since I didn’t update the docker file, the merge there is actually needed to see whether the benchmarking of ReBenchDB with ReBench itself works.

Signed-off-by: Stefan Marr <[email protected]>