From 65f6b44b2a0759fa6c950e8646d2e8aed0937aab Mon Sep 17 00:00:00 2001 From: Kostas Krikellas <131142368+kkrik-es@users.noreply.github.com> Date: Wed, 12 Feb 2025 15:07:38 +0200 Subject: [PATCH] Handle error in concurrent downsample actions (#122251) --- muted-tests.yml | 3 --- .../xpack/downsample/DownsampleActionSingleNodeTests.java | 4 ++-- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/muted-tests.yml b/muted-tests.yml index 9fc54786d579..1f42941579cf 100644 --- a/muted-tests.yml +++ b/muted-tests.yml @@ -360,9 +360,6 @@ tests: - class: org.elasticsearch.test.rest.ClientYamlTestSuiteIT method: test {yaml=snapshot.delete/10_basic/Delete a snapshot asynchronously} issue: https://github.com/elastic/elasticsearch/issues/122102 -- class: org.elasticsearch.xpack.downsample.DownsampleActionSingleNodeTests - method: testDuplicateDownsampleRequest - issue: https://github.com/elastic/elasticsearch/issues/122158 - class: org.elasticsearch.search.SearchCancellationIT method: testCancelFailedSearchWhenPartialResultDisallowed issue: https://github.com/elastic/elasticsearch/issues/121719 diff --git a/x-pack/plugin/downsample/src/test/java/org/elasticsearch/xpack/downsample/DownsampleActionSingleNodeTests.java b/x-pack/plugin/downsample/src/test/java/org/elasticsearch/xpack/downsample/DownsampleActionSingleNodeTests.java index ce9b60938526..2c759f150e57 100644 --- a/x-pack/plugin/downsample/src/test/java/org/elasticsearch/xpack/downsample/DownsampleActionSingleNodeTests.java +++ b/x-pack/plugin/downsample/src/test/java/org/elasticsearch/xpack/downsample/DownsampleActionSingleNodeTests.java @@ -1760,7 +1760,7 @@ public class DownsampleActionSingleNodeTests extends ESSingleNodeTestCase { new Thread(() -> { try { downsample(sourceIndex, targetIndex, config); - } catch (ResourceAlreadyExistsException e) { + } catch (ElasticsearchException e) { firstFailed.set(true); } finally { downsampleComplete.countDown(); @@ -1770,7 +1770,7 @@ public class DownsampleActionSingleNodeTests extends ESSingleNodeTestCase { new Thread(() -> { try { downsample(sourceIndex, targetIndex, config); - } catch (ResourceAlreadyExistsException e) { + } catch (ElasticsearchException e) { secondFailed.set(true); } finally { downsampleComplete.countDown();