Skip to content

Commit

Permalink
[ML] unmuting test for testTooManyPartitions memory check on windows (e…
Browse files Browse the repository at this point in the history
…lastic#62393) (elastic#62405)

This commit unmutes the windows check for testTooManyPartitions test.

The assertion has since changed to include a soft_limit check.

This coupled with changes over the past years means the test should be enabled again.

related to: elastic#32033
  • Loading branch information
benwtrent authored Sep 16, 2020
1 parent 6a016fb commit 8d89a28
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
*/
package org.elasticsearch.xpack.ml.integration;

import org.apache.lucene.util.Constants;
import org.elasticsearch.common.unit.TimeValue;
import org.elasticsearch.xpack.core.ml.action.GetJobsStatsAction;
import org.elasticsearch.xpack.core.ml.job.config.AnalysisConfig;
Expand Down Expand Up @@ -40,7 +39,6 @@ public void cleanUpTest() {

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/43013")
public void testTooManyPartitions() throws Exception {
assumeFalse("AwaitsFix(bugUrl = \"https://github.com/elastic/elasticsearch/issues/32033\")", Constants.WINDOWS);
Detector.Builder detector = new Detector.Builder("count", null);
detector.setPartitionFieldName("user");

Expand Down

0 comments on commit 8d89a28

Please sign in to comment.