Skip to content

[ES|QL] Changes the ccs flag#236030

Merged
stratoula merged 3 commits intoelastic:mainfrom
stratoula:change-ccs-flag
Sep 24, 2025
Merged

[ES|QL] Changes the ccs flag#236030
stratoula merged 3 commits intoelastic:mainfrom
stratoula:change-ccs-flag

Conversation

@stratoula
Copy link
Contributor

@stratoula stratoula commented Sep 23, 2025

Summary

ES is adding a new parameter for retrieving ccs metadata. This works the same as the include_ccs_metadata but they want us to use the new one instead elastic/elasticsearch#134446. The reasons they did this change are:

  • they are trying to avoid the "ccs" term in serverless as much as possible
  • to be consistent with the flat world model of "all projects should behave the same in a search", in the future we might be changing include_execution_metadata to also return info for local-only searches, which it doesn't do right now.

Both params will be available in serverless and stateful
@stratoula stratoula added v9.2.0 Feature:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// labels Sep 23, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 24, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #97 / Endpoint plugin @ess @serverless @skipInServerlessMKI Endpoint execute response action "after all" hook for "should succeed with valid endpoint id, command and an optional timeout"
  • [job] [logs] Jest Tests #5 / MonitorEditPage shows duplicate error when "nameAlreadyExists" is false
  • [job] [logs] Jest Tests #5 / MonitorEditPage shows duplicate error when "nameAlreadyExists" is true

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
data 423.4KB 423.4KB +6.0B

History
@stratoula stratoula marked this pull request as ready for review September 24, 2025 09:18
@stratoula stratoula requested review from a team as code owners September 24, 2025 09:18
@stratoula stratoula requested a review from a team September 24, 2025 09:18
@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 24, 2025

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)
@stratoula stratoula merged commit e1f77c8 into elastic:main Sep 24, 2025
13 checks passed
rbrtj pushed a commit to rbrtj/kibana that referenced this pull request Sep 25, 2025
@stratoula @rbrtj
[ES|QL] Changes the ccs flag (elastic#236030)
38a9e63 
## Summary

ES is adding a new parameter for retrieving ccs metadata. This works the
same as the `include_ccs_metadata` but they want us to use the new one
instead elastic/elasticsearch#134446. The
reasons they did this change are:

- they are trying to avoid the "ccs" term in serverless as much as
possible
- to be consistent with the flat world model of "all projects should
behave the same in a search", in the future we might be changing
include_execution_metadata to also return info for local-only searches,
which it doesn't do right now.

Both params will be available in serverless and stateful
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Sep 25, 2025
@stratoula @CoenWarmer
[ES|QL] Changes the ccs flag (elastic#236030)
e993dbf 
## Summary

ES is adding a new parameter for retrieving ccs metadata. This works the
same as the `include_ccs_metadata` but they want us to use the new one
instead elastic/elasticsearch#134446. The
reasons they did this change are:

- they are trying to avoid the "ccs" term in serverless as much as
possible
- to be consistent with the flat world model of "all projects should
behave the same in a search", in the future we might be changing
include_execution_metadata to also return info for local-only searches,
which it doesn't do right now.

Both params will be available in serverless and stateful
niros1 pushed a commit that referenced this pull request Sep 30, 2025
@stratoula @niros1
[ES|QL] Changes the ccs flag (#236030)
d17cb51 
## Summary

ES is adding a new parameter for retrieving ccs metadata. This works the
same as the `include_ccs_metadata` but they want us to use the new one
instead elastic/elasticsearch#134446. The
reasons they did this change are:

- they are trying to avoid the "ccs" term in serverless as much as
possible
- to be consistent with the flat world model of "all projects should
behave the same in a search", in the future we might be changing
include_execution_metadata to also return info for local-only searches,
which it doesn't do right now.

Both params will be available in serverless and stateful
rylnd pushed a commit to rylnd/kibana that referenced this pull request Oct 17, 2025
@stratoula
[ES|QL] Changes the ccs flag (elastic#236030)
539234d 
## Summary

ES is adding a new parameter for retrieving ccs metadata. This works the
same as the `include_ccs_metadata` but they want us to use the new one
instead elastic/elasticsearch#134446. The
reasons they did this change are:

- they are trying to avoid the "ccs" term in serverless as much as
possible
- to be consistent with the flat world model of "all projects should
behave the same in a search", in the future we might be changing
include_execution_metadata to also return info for local-only searches,
which it doesn't do right now.

Both params will be available in serverless and stateful
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// v9.2.0

4 participants

@stratoula @elasticmachine @dgieselaar @jughosta