Skip to content

Conversation

@jihyunjeongme
Copy link

Description

This PR Add typed_keys parameter set as true:

  • a JsonException that occurs when using aggregations with SearchTemplateAsync because the mandatory typed_keys=true parameter is undocumented.
  • This change improves the developer experience by preventing user confusion and saving significant debugging time when using search templates.

Related issues

### Description
This PR Add `typed_keys` parameter set as `true`:
- a `JsonException` that occurs when using aggregations with `SearchTemplateAsync` because the mandatory `typed_keys=true` parameter is undocumented.
- This change improves the developer experience by preventing user confusion and saving significant debugging time when using search templates.

### Related issues
- #7605 (comment)
- elastic/sdh-search#1648 (comment)
@flobernd
Copy link
Member

Hi @jihyunjeongme thanks a lot for creating this PR. Much appreciated 🙂

We checked again and it seems that we can safely set typed_keys to true in this case as well. I will implement this change and close this PR in favor of a fix when I'm done.

@flobernd
Copy link
Member

Closed in favor of #8609

@flobernd flobernd closed this Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants