Skip to content

Commit defd164

Browse files
committed
Handle error in concurrent downsample actions
1 parent 28f51e4 commit defd164

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -381,9 +381,6 @@ tests:
381381
- class: org.elasticsearch.xpack.ml.integration.ClassificationIT
382382
method: testWithOnlyTrainingRowsAndTrainingPercentIsFifty_DependentVariableIsBoolean
383383
issue: https://github.com/elastic/elasticsearch/issues/121680
384-
- class: org.elasticsearch.xpack.downsample.DownsampleActionSingleNodeTests
385-
method: testDuplicateDownsampleRequest
386-
issue: https://github.com/elastic/elasticsearch/issues/122158
387384
- class: org.elasticsearch.search.SearchCancellationIT
388385
method: testCancelFailedSearchWhenPartialResultDisallowed
389386
issue: https://github.com/elastic/elasticsearch/issues/121719

x-pack/plugin/downsample/src/test/java/org/elasticsearch/xpack/downsample/DownsampleActionSingleNodeTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1760,7 +1760,7 @@ public void testDuplicateDownsampleRequest() throws Exception {
17601760
new Thread(() -> {
17611761
try {
17621762
downsample(sourceIndex, targetIndex, config);
1763-
} catch (ResourceAlreadyExistsException e) {
1763+
} catch (ElasticsearchException e) {
17641764
firstFailed.set(true);
17651765
} finally {
17661766
downsampleComplete.countDown();
@@ -1770,7 +1770,7 @@ public void testDuplicateDownsampleRequest() throws Exception {
17701770
new Thread(() -> {
17711771
try {
17721772
downsample(sourceIndex, targetIndex, config);
1773-
} catch (ResourceAlreadyExistsException e) {
1773+
} catch (ElasticsearchException e) {
17741774
secondFailed.set(true);
17751775
} finally {
17761776
downsampleComplete.countDown();

0 commit comments

Comments
 (0)