Skip to content

Fixing random_sampler tests #83549

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 7, 2022

Conversation

benwtrent
Copy link
Member

Fixes smoke tests my enabling the random_sampler agg

Fixes flakey sampler test

closes #83523

closes #83357

Fixes smoke tests my enabling the random_sampler agg

Fixes flakey sampler test

closes elastic#83523

closes elastic#83357
@benwtrent benwtrent added >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations v8.1.0 v8.2.0 labels Feb 4, 2022
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 4, 2022
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@dimitris-athanasiou dimitris-athanasiou left a comment

Choose a reason for hiding this comment

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

LGTM

@benwtrent benwtrent merged commit 73b323e into elastic:master Feb 7, 2022
@benwtrent benwtrent deleted the feature/sampling-test-fixes branch February 7, 2022 13:34
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.1 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 83549

benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Feb 7, 2022
Fixes smoke tests my enabling the random_sampler agg

Fixes flakey sampler test

closes elastic#83523

closes elastic#83357
elasticsearchmachine pushed a commit that referenced this pull request Feb 7, 2022
Fixes smoke tests my enabling the random_sampler agg

Fixes flakey sampler test

closes #83523

closes #83357
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Feb 7, 2022
* upstream/master:
  [DOCS] Switch xrefs to external links (elastic#83590)
  [DOCS] 'features' flag added in elastic#83083 (elastic#83452)
  Rename ChangePolicyforIndexIT to ChangePolicyForIndexIT (elastic#83569)
  Fixing random_sampler tests (elastic#83549)
  Upgrade Checkstyle to 9.3 (elastic#83314)
  Make improvements to the release notes generator (elastic#83525)
  Cleanup DataTierAllocationDecider (elastic#83572)
  Upgrade jANSI dependency to 2.4.0 (elastic#83566)
  Speed up Name Collision Check in Metadata.Builder (elastic#83340)
  SQL: Add range checks to interval multiplication operation (elastic#83478)
  Remove DiscoveryNodes#getAllNodes (elastic#83538)
  Make RoutingNodes behave like a collection (elastic#83540)
  Remove Unused CS Listener from SecurityServerTransportInterceptor (elastic#83556)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.1.0 v8.2.0
Projects
None yet
5 participants