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.

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

Benchmark for date_histo with one bucket #165

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Apr 1, 2021

This adds a benchmark for date_histogram when a single bucket covers
the entire index. We expect this to be quite common for buckets with a
size near a day. Its fairly common for folks to use "daily" indices that
contain about a day's worth of data. That day isn't likely to line up
exactly with daily buckets. But for buckets marginally larger than a
day and for indices even a little smaller than a day I think it'll be
quite common.

Benchmark for elastic/elasticsearch#71180

This adds a benchmark for `date_histogram` when a single bucket covers
the entire index. We expect this to be quite common for buckets with a
size near a day. Its fairly common for folks to use "daily" indices that
contain *about* a day's worth of data. That day isn't likely to line up
*exactly* with daily buckets. But for buckets marginally larger than a
day and for indices even a little smaller than a day I think it'll be
quite common.
@dliappis dliappis self-requested a review April 2, 2021 10:05
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM.

@dliappis
Copy link
Contributor

dliappis commented Apr 2, 2021

@nik9000 will merge this on Monday (typically we won't merge track changes on Fri to prevent chances of broken nightlies during the weekend). Question: how far back should this be backported?

@nik9000
Copy link
Member Author

nik9000 commented Apr 2, 2021

Thanks!

@dliappis dliappis merged commit 0c72b79 into elastic:master Apr 5, 2021
@dliappis
Copy link
Contributor

dliappis commented Apr 5, 2021

@nik9000 regarding backporting, how far back can this go?

@nik9000
Copy link
Member Author

nik9000 commented Apr 5, 2021 via email

@dliappis
Copy link
Contributor

dliappis commented Apr 5, 2021

The code should work on most versions of es after .say, 1.0.

Do you mean after 7.12.0 or?

@nik9000
Copy link
Member Author

nik9000 commented Apr 5, 2021

Do you mean after 7.12.0 or?

I mean its safe to backport it as far as you like. But 7.12 is certainly far enough for me. 7.13 even.

dliappis pushed a commit that referenced this pull request Apr 5, 2021
This adds a benchmark for `date_histogram` when a single bucket covers
the entire index. We expect this to be quite common for buckets with a
size near a day. Its fairly common for folks to use "daily" indices that
contain *about* a day's worth of data. That day isn't likely to line up
*exactly* with daily buckets. But for buckets marginally larger than a
day and for indices even a little smaller than a day I think it'll be
quite common.
@dliappis
Copy link
Contributor

dliappis commented Apr 5, 2021

I mean its safe to backport it as far as you like. But 7.12 is certainly far enough for me. 7.13 even.

Backported to 7.12 using 11cee00

@nik9000
Copy link
Member Author

nik9000 commented Apr 5, 2021

Thanks!

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