Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support term aggregation on non text fields #1689

Open
6 of 7 tasks
PSeitz opened this issue Nov 22, 2022 · 2 comments
Open
6 of 7 tasks

Support term aggregation on non text fields #1689

PSeitz opened this issue Nov 22, 2022 · 2 comments
Assignees
@PSeitz
Labels
high priority project airmail

Comments

@PSeitz
Copy link
Contributor

PSeitz commented Nov 22, 2022 •
edited
Loading

Term aggregation only works on text fields currently.
Extend support for (single and multi-value):

Bucket Key type

u64

  • u64
  • i64
  • f64
  • boolean
  • DateTime

&[u8]

  • bytes

u128

  • ip address
@PSeitz PSeitz mentioned this issue Feb 20, 2023
Merged
@PSeitz PSeitz mentioned this issue May 15, 2023
Merged
@PSeitz PSeitz self-assigned this Jun 7, 2023
@PSeitz PSeitz mentioned this issue Sep 6, 2023
Merged
@fulmicoton
Copy link
Collaborator

fulmicoton commented Feb 14, 2024 •
edited
Loading

ETA on bool / ipvaddr? (I don't think we care about bytes all that much)

@PSeitz
Copy link
Contributor Author

PSeitz commented Feb 14, 2024

bool should be simple
for ip addresses we need to change the fast field API to get a u64 representation for the u128 values (which may limit possible codecs), to reuse the aggregation code

@PSeitz PSeitz mentioned this issue Feb 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PSeitz PSeitz

Labels
high priority project airmail
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fulmicoton @PSeitz