Skip to content

Conversation

smalyshev
Copy link
Contributor

It duplicates CrossClusterQueryWithPartialResultsIT so there's no more need of it.

Fixes #122336

It duplicates CrossClusterQueryWithPartialResultsIT so there's no more need of it.
@smalyshev smalyshev added >test Issues or PRs that are addressing/adding tests :Search Foundations/CCS auto-backport Automatically create backport pull requests when merged v9.1.0 v8.19.0 labels Feb 20, 2025
@smalyshev smalyshev requested review from dnhatn and quux00 February 20, 2025 17:47
@smalyshev smalyshev marked this pull request as ready for review February 20, 2025 17:47
@elasticsearchmachine elasticsearchmachine added the Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch label Feb 20, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-foundations (Team:Search Foundations)

Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@smalyshev smalyshev enabled auto-merge (squash) February 20, 2025 18:20
@smalyshev smalyshev merged commit 15474a2 into elastic:main Feb 20, 2025
17 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 123054

smalyshev added a commit to smalyshev/elasticsearch that referenced this pull request Feb 20, 2025
* Remove testCloseSkipUnavailable It duplicates CrossClusterQueryWithPartialResultsIT so there's no more need of it. (cherry picked from commit 15474a2) # Conflicts: #	muted-tests.yml
@smalyshev
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Feb 20, 2025
* Remove testCloseSkipUnavailable It duplicates CrossClusterQueryWithPartialResultsIT so there's no more need of it. (cherry picked from commit 15474a2) # Conflicts: #	muted-tests.yml
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request Feb 21, 2025
* Remove testCloseSkipUnavailable It duplicates CrossClusterQueryWithPartialResultsIT so there's no more need of it.
JeremyDahlgren added a commit to JeremyDahlgren/elasticsearch that referenced this pull request Aug 14, 2025
createLocalIndex() was added in elastic#121240 but is unused now after testCloseSkipUnavailable() was removed in elastic#123054.
JeremyDahlgren added a commit that referenced this pull request Aug 14, 2025
createLocalIndex() was added in #121240 but is unused now after testCloseSkipUnavailable() was removed in #123054.
joshua-adams-1 pushed a commit to joshua-adams-1/elasticsearch that referenced this pull request Aug 15, 2025
createLocalIndex() was added in elastic#121240 but is unused now after testCloseSkipUnavailable() was removed in elastic#123054.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Search Foundations/CCS Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test Issues or PRs that are addressing/adding tests v8.19.0 v9.1.0

3 participants