Skip to content

Commit

Permalink
Fix flaky test in CardinalityWithRequestBreakerIT
Browse files Browse the repository at this point in the history
Signed-off-by: Neetika Singhal <neetiks@amazon.com>
  • Loading branch information
neetikasinghal committed Dec 5, 2023
1 parent f7f3500 commit f4275aa
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@
import org.opensearch.indices.breaker.HierarchyCircuitBreakerService;
import org.opensearch.search.aggregations.Aggregator;
import org.opensearch.search.aggregations.BucketOrder;
import org.opensearch.test.OpenSearchIntegTestCase;
import org.opensearch.test.ParameterizedOpenSearchIntegTestCase;

import java.util.Arrays;
Expand All @@ -54,6 +55,7 @@
import static org.opensearch.search.aggregations.AggregationBuilders.cardinality;
import static org.opensearch.search.aggregations.AggregationBuilders.terms;

@OpenSearchIntegTestCase.ClusterScope(scope = OpenSearchIntegTestCase.Scope.SUITE)
public class CardinalityWithRequestBreakerIT extends ParameterizedOpenSearchIntegTestCase {

public CardinalityWithRequestBreakerIT(Settings dynamicSettings) {
Expand All @@ -76,7 +78,6 @@ protected Settings featureFlagSettings() {
/**
* Test that searches using cardinality aggregations returns all request breaker memory.
*/
@AwaitsFix(bugUrl = "https://github.com/opensearch-project/OpenSearch/issues/10154")
public void testRequestBreaker() throws Exception {
final String requestBreaker = randomIntBetween(1, 10000) + "kb";
logger.info("--> Using request breaker setting: {}", requestBreaker);
Expand Down

0 comments on commit f4275aa

Please sign in to comment.