Skip to content

Commit

Permalink
Expose RawCohortResult.cohort
Browse files Browse the repository at this point in the history
  • Loading branch information
dagguh committed Dec 11, 2023
1 parent 88a7f94 commit fc111e8
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,9 @@ Dropping a requirement of a major version of a dependency is a new contract.
## [Unreleased]
[Unreleased]: https://github.com/atlassian/report/compare/release-4.1.0...master

### Added
- Expose `RawCohortResult.cohort`.

## [4.1.0] - 2023-11-30
[4.1.0]: https://github.com/atlassian/report/compare/release-4.0.0...release-4.1.0

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,11 @@ abstract class RawCohortResult private constructor() {
*/
abstract val failure: Exception?

/**
* Textual description of the cohort, e.g. "9.12.0 on DC 2 nodes - experiment, sample 3".
*/
abstract val cohort: String

/**
* Prepares post-processed performance results.
*/
Expand Down Expand Up @@ -68,7 +73,7 @@ abstract class RawCohortResult private constructor() {
}

private class FullRawCohortResult(
private val cohort: String,
override val cohort: String,
override val results: Path,
format: MetricJsonFormat
) : RawCohortResult() {
Expand Down Expand Up @@ -113,7 +118,7 @@ abstract class RawCohortResult private constructor() {
}

private class FailedRawCohortResult(
private val cohort: String,
override val cohort: String,
override val results: Path,
override val failure: Exception
) : RawCohortResult() {
Expand All @@ -134,7 +139,7 @@ abstract class RawCohortResult private constructor() {
}

private class LegacyRawCohortResult(
private val cohort: String,
override val cohort: String,
override val failure: Exception
) : RawCohortResult() {

Expand Down

0 comments on commit fc111e8

Please sign in to comment.