Skip to content

Commit bb66699

Browse files
author
Peter Alfonsi
committed
Attempt to fix broken ITs
Signed-off-by: Peter Alfonsi <[email protected]>
1 parent 58170bf commit bb66699

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

server/src/internalClusterTest/java/org/opensearch/indices/IndicesRequestCacheIT.java

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -81,18 +81,14 @@ public static Collection<Object[]> parameters() {
8181
new Object[] { Settings.builder().put(CLUSTER_CONCURRENT_SEGMENT_SEARCH_SETTING.getKey(), true).build() }
8282
);
8383
}
84-
/*
85-
new Object[] { Settings.builder().put(FeatureFlags.PLUGGABLE_CACHE, "true").build() },
86-
new Object[] { Settings.builder().put(FeatureFlags.PLUGGABLE_CACHE, "false").build() }
87-
*/
8884

8985
@Override
9086
protected Settings featureFlagSettings() {
91-
return Settings.builder()
92-
.put(super.featureFlagSettings()).put(FeatureFlags.CONCURRENT_SEGMENT_SEARCH, "true")
93-
.put(super.featureFlagSettings()).put(FeatureFlags.PLUGGABLE_CACHE, "true")
87+
return Settings.builder().put(super.featureFlagSettings())
88+
.put(FeatureFlags.CONCURRENT_SEGMENT_SEARCH, "true")
89+
.put(FeatureFlags.PLUGGABLE_CACHE, "true")
9490
.build();
95-
}
91+
} // For now hardcode TC feature flag as true. Attempt to backport the changes allowing us to parameterize it
9692

9793
// One of the primary purposes of the query cache is to cache aggs results
9894
public void testCacheAggs() throws Exception {

0 commit comments

Comments
 (0)