Add an opt-in partial-result mode for aggregations on text/keyword mapping conflicts - #5657

Merged

ahkcs merged 32 commits into

opensearch-project:mainfrom

ahkcs:feature/ppl-partial-result-warning-channel

Sep 1, 2026

Commits on Sep 1, 2026

Add non-fatal warning channel to PPL query response
ahkcs
committed
83fd850
Return a partial result instead of exhausting PIT on a mapping conflict
ahkcs
committed
f8330d5
Gate partial results on a warning-capable response format
ahkcs
committed
a6085e6
Flatten per-index mappings when partitioning for partial results
ahkcs
committed
834b156
Refine partial-result index selection and warning wording
ahkcs
committed
de789c1
Truncate the excluded-index list in the partial-result warning
ahkcs
committed
1262d8e
Fix partial-result warning wording to reflect the pushdown criterion
ahkcs
committed
66367c0
Extract partial-result partitioning into its own class with unit tests
ahkcs
committed
208589d
Allow a per-request override for partial-result mode
ahkcs
committed
a3fa7b9
Simplify partial-result warning to name the excluded indices and the fix
ahkcs
committed
960984b
Do not resolve response format for explain requests
ahkcs
committed
a5759ff
Cover the null-warnings branch in QueryResult to satisfy protocol coverage
ahkcs
committed
090dd39
Address review: rename the partial-result setting and fold the fallback into pushDownAggregate
ahkcs
committed
60d3a53
Reuse the already-fetched index mappings for partial-result partitioning
ahkcs
committed
f50c008
Fix formatting of the renamed partial-result setting key
ahkcs
committed
4216040
Revert the index-mapping reuse optimization: it exposed a merge-mutation bug
ahkcs
committed
11ae75e
Reuse the already-fetched index mappings, and stop the merge mutating them
ahkcs
committed
3da0b58
Clear the per-request partial-result state after each query
ahkcs
committed
05a78d0
Update explain golden files for the changed OpenSearchDataType serialVersionUID
ahkcs
committed
36e3474
Decide partial-result mode before pushdown analysis, not after it fails
ahkcs
committed
b30124b
Document the partial-result-on-mapping-conflict setting
ahkcs
committed
091eebd
Tighten inline comments on the partial-result path
ahkcs
committed
6037584
Address review: mark setting experimental, consolidate enable check
ahkcs
committed
5220471
Drop the redundant post-failure partial-result fallbacks
ahkcs
committed
8a722ac
Tighten inline comments
ahkcs
committed
2a9058a
Bail on a non-text type conflict instead of excluding the index
ahkcs
committed
1904867
Partition partial results on the group key's source fields
ahkcs
committed
fc803bd
Add IT for a multi-field expression group key
ahkcs
committed
0a9020c
Handle a non-text type vs text conflict in partial results
ahkcs
committed
b89193f
Revert non-text type generalization; scope to text/keyword collapse
ahkcs
committed
009cd0b
Generalize partial results to any non-aggregatable-vs-aggregatable conflict
ahkcs
committed
97aa640
Address review: drop the special keyword token; sort only when building the warning
ahkcs
committed
d647281

Navigation Menu

Uh oh!

Add an opt-in partial-result mode for aggregations on text/keyword mapping conflicts - #5657

Add an opt-in partial-result mode for aggregations on text/keyword mapping conflicts#5657
ahkcs merged 32 commits into
opensearch-project:mainfrom
ahkcs:feature/ppl-partial-result-warning-channel

Commits on Sep 1, 2026

Add non-fatal warning channel to PPL query response

Return a partial result instead of exhausting PIT on a mapping conflict

Gate partial results on a warning-capable response format

Flatten per-index mappings when partitioning for partial results

Refine partial-result index selection and warning wording

Truncate the excluded-index list in the partial-result warning

Fix partial-result warning wording to reflect the pushdown criterion

Extract partial-result partitioning into its own class with unit tests

Allow a per-request override for partial-result mode

Simplify partial-result warning to name the excluded indices and the fix

Do not resolve response format for explain requests

Cover the null-warnings branch in QueryResult to satisfy protocol coverage

Address review: rename the partial-result setting and fold the fallback into pushDownAggregate

Reuse the already-fetched index mappings for partial-result partitioning

Fix formatting of the renamed partial-result setting key

Revert the index-mapping reuse optimization: it exposed a merge-mutation bug

Reuse the already-fetched index mappings, and stop the merge mutating them

Clear the per-request partial-result state after each query

Update explain golden files for the changed OpenSearchDataType serialVersionUID

Decide partial-result mode before pushdown analysis, not after it fails

Document the partial-result-on-mapping-conflict setting

Tighten inline comments on the partial-result path

Address review: mark setting experimental, consolidate enable check

Drop the redundant post-failure partial-result fallbacks

Tighten inline comments

Bail on a non-text type conflict instead of excluding the index

Partition partial results on the group key's source fields

Add IT for a multi-field expression group key

Handle a non-text type vs text conflict in partial results

Revert non-text type generalization; scope to text/keyword collapse

Generalize partial results to any non-aggregatable-vs-aggregatable conflict

Address review: drop the special keyword token; sort only when building the warning