Skip to content

Conversation

@csoulios
Copy link
Contributor

Backport the AggregatorTestCase changes implemented in #52851

Also, this PR modifies random half float generation so that it respects the half float limits.

Finally it fixes #55299

@csoulios csoulios added >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations backport v7.8.0 labels Apr 16, 2020
@csoulios csoulios requested a review from polyfractal April 16, 2020 15:43
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (:Analytics/Aggregations)

Copy link
Contributor

@polyfractal polyfractal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@csoulios csoulios merged commit b810f00 into elastic:7.x Apr 16, 2020
@csoulios csoulios deleted the backport-aggtestcase-writeTestDoc branch April 16, 2020 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations backport >test Issues or PRs that are addressing/adding tests v7.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] MedianAbsoluteDeviationAggregatorTests testSupportedFieldTypes failure

3 participants