Repository navigation
Case and Coalesce Support for Compound Types - #140677
not-napoleon merged 17 commits into
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?
|
|
I'm confidant that this is not going to break the release tests, so I'm taking that tag off now. |
This PR adds case function support for the
HISTOGRAM,TDIGEST, andAGGREGATE_METRIC_DOUBLEtypes. No changes to the actual behavior ofCasewere necessary, just updating the type checking to allow for these types. I've also added CSV tests for these, which is the bulk of the PR and involved adding some additional data to the CSV data files.I've also added Coalesce support for t-digest and histogram. Aggregate metric double needs a little more work in the block loader I think (or at least at first glance), so I didn't do that here. That's probably worth thinking about at some point down the road.
Since this introduces new CSV tests, I added a capability to gate those tests and I'll tag the PR for release testing to minimize the chances of breaking the release builds.