Release histogram data type#139703
Conversation
|
Pinging @elastic/es-storage-engine (Team:StorageEngine) |
|
Hi @not-napoleon, I've created a changelog YAML for you. |
ℹ️ Important: Docs version tagging👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version. We use applies_to tags to mark version-specific features and changes. Expand for a quick overviewWhen to use applies_to tags:✅ At the page level to indicate which products/deployments the content applies to (mandatory) What NOT to do:❌ Don't remove or replace information that applies to an older version 🤔 Need help?
|
| /** | ||
| * Release version for Histogram data type support | ||
| */ | ||
| public static final TransportVersion ESQL_HISTOGRAM_DATATYPE_RELEASE = TransportVersion.fromName("esql_histogram_datatype_release"); |
There was a problem hiding this comment.
Should we call it tech preview for now?
There was a problem hiding this comment.
The transport version is internal use only, and as far as I can tell this will be the "supported since" version forever, so "release" made more sense to me. I can change it if you really want me to though.
There was a problem hiding this comment.
No need to touch it here, maybe something to consider - to keep it in lockstep with exponential and tdigest.
|
After conferring with @kkrik-es , we are bypassing CI and merging this. The failures look unrelated, and we want to get this in today. |
This takes the Histogram data type out from under construction and makes it available in release builds. This includes basic support for scalar and aggregation functions that operate on all types (except for Count, Coalesce, and Case, which are going to need special handling later). In conjunction with #139637, this will allow aggregating histogram fields as t-digests to compute percentiles and other statistics.