Skip to content

Conversation

@rdettai-sk
Copy link
Collaborator

@rdettai-sk rdettai-sk commented Dec 23, 2025

Description

Clarify how json types are stored and provide some pointers regarding concat field specificities.

@rdettai-sk rdettai-sk force-pushed the imp-messy-type-docs branch 2 times, most recently from d92eda1 to 20c4d44 Compare December 23, 2025 15:18
- numeric values are cast to the first compatible format between `i64`, `u64` or
`f64` (in this order)
- for string values (surrounded with quotes), Tantivy attempts to parse a date
in `rfc3339` format. If the parsing fails, the value is stored as `text` using
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm not sure that's what is actually happening, following this thread #6074 (comment)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For regular json types (and dynamic mappings of course) it is. It's only concatenate fields where the JSON text value doesn't go through this parsing. I clarified this now.

@rdettai-sk rdettai-sk changed the title Improve messy type documentation Improve documentation regarding json type and concatenated json Jan 6, 2026
@rdettai-sk rdettai-sk force-pushed the imp-messy-type-docs branch from 9ee2d8e to 91113c3 Compare January 6, 2026 14:54
@rdettai-sk rdettai-sk force-pushed the imp-messy-type-docs branch 2 times, most recently from 683c53d to 196dcab Compare January 6, 2026 14:57
@rdettai-sk rdettai-sk force-pushed the imp-messy-type-docs branch from 196dcab to 68c2ea2 Compare January 6, 2026 14:59
@trinity-1686a trinity-1686a enabled auto-merge (squash) January 7, 2026 09:40
@trinity-1686a trinity-1686a merged commit bdcdd7d into main Jan 7, 2026
8 checks passed
@trinity-1686a trinity-1686a deleted the imp-messy-type-docs branch January 7, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants