Skip to content

Comments

ES|QL: Release CCS support for FORK#139630

Merged
ioanatia merged 4 commits intoelastic:mainfrom
ioanatia:enable_css_fork
Dec 17, 2025
Merged

ES|QL: Release CCS support for FORK#139630
ioanatia merged 4 commits intoelastic:mainfrom
ioanatia:enable_css_fork

Conversation

@ioanatia
Copy link
Contributor

Release CCS support for FORK.

CCS support for FORK and subqueries was fixed with #138332 and we have not seen any failures since we enabled the tests with #139302.

@ioanatia ioanatia added >enhancement Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch :Search Relevance/ES|QL Search functionality in ES|QL labels Dec 16, 2025
@elasticsearchmachine
Copy link
Collaborator

Hi @ioanatia, I've created a changelog YAML for you.

@ioanatia ioanatia marked this pull request as ready for review December 16, 2025 18:10
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

Copy link
Member

@fang-xing-esql fang-xing-esql left a comment

Choose a reason for hiding this comment

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

The change in ComputeService LGTM!

@ioanatia ioanatia mentioned this pull request Dec 16, 2025
17 tasks
Copy link
Member

@carlosdelest carlosdelest left a comment

Choose a reason for hiding this comment

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

👍


return input -> {
if (EsqlCapabilities.Cap.ENABLE_FORK_FOR_REMOTE_INDICES_V2.isEnabled() == false) {
checkForRemoteClusters(input, source(ctx), "FORK");
Copy link
Member

Choose a reason for hiding this comment

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

I think we can remove checkForRemoteClusters() now?

@ioanatia ioanatia merged commit f7364e1 into elastic:main Dec 17, 2025
35 checks passed
@ioanatia ioanatia deleted the enable_css_fork branch December 17, 2025 09:48
szybia added a commit to szybia/elasticsearch that referenced this pull request Dec 17, 2025
…-err-message

* upstream/main: (45 commits)
  Add sort field usage to telemetry (elastic#139530)
  ES|QL: Release CCS support for FORK (elastic#139630)
  Parameterize VectorSimilarityFunctionsTests on the similarity function (elastic#139516)
  fix broken links from ccs file move (elastic#139655)
  Update docs for v9.2.3 release (elastic#139479)
  Move tsdb bwc tests to x-pack/logsdb (elastic#139671)
  Fix FirstDocIdGroupingAggregatorFunction (elastic#139619)
  Fix release test for node_reduction profiling (elastic#139515)
  Unmute RestClientSingleHostIntegTests.testRequestResetAndAbort (elastic#139656)
  Unmute VerifyVersionConstantsIT.testLuceneVersionConstant (elastic#139644)
  Mute org.elasticsearch.repositories.gcs.GoogleCloudStorageBlobStoreRepositoryTests testReadNonExistingPath elastic#139665
  Mute org.elasticsearch.smoketest.WatcherYamlRestIT test {p0=mustache/10_webhook/Test webhook action with mustache integration} elastic#139663
  [ES|QL] Run aggregations on aggregate metric double with default metric (elastic#138647)
  Enable TDigest field mapper and ES|QL type (elastic#139607)
  Mute org.elasticsearch.index.mapper.HalfFloatSyntheticSourceNativeArrayIntegrationTests testSynthesizeArrayRandom elastic#139658
  Explicitly cleanup test index with shared data path (elastic#136048)
  Mute org.elasticsearch.xpack.core.action.XPackUsageResponseTests testVersionDependentSerializationWriteToOldStream elastic#139576
  Make XPackUsageResponseTests a wire serializing test case (elastic#139643)
  Mute org.elasticsearch.upgrades.UpgradeClusterClientYamlTestSuiteIT test {p0=mixed_cluster/90_ml_data_frame_analytics_crud/Start and stop old regression job} elastic#139654
  Relax error bounds for RandomizedTimeSeriesIT (elastic#139641)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>enhancement :Search Relevance/ES|QL Search functionality in ES|QL Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants