Skip to content

Conversation

@SAKavli
Copy link
Contributor

@SAKavli SAKavli commented Aug 14, 2025

We wish to query for entries which have logged both the experiment type and num params. However, if logs gets too long, they get divided into chunks, which might separate experiment type and num params into different log chunks, which complicates the query logic.

This change will ensure the first chunk will always contain both of these properties.

(cherry picked from commit c0d5c5f)

Manual Backport PR

We wish to query for entries which have logged both the experiment type
and num params. However, if logs gets too long, they get divided into
chunks, which might separate experiment type and num params
into different log chunks, which complicates the query logic.

This change will ensure the first chunk will always contain both
of these properties.

(cherry picked from commit c0d5c5f)
@codspeed-hq
Copy link

codspeed-hq bot commented Aug 14, 2025

CodSpeed Performance Report

Merging #11460 will not alter performance

Comparing SAKavli:backport-c0d5c-into-14.4 (3cd41e0) with main (d39b225)1

Summary

✅ 22 untouched benchmarks

Footnotes

  1. No successful run was found on version-14.4 (926f09a) during the generation of this report, so main (d39b225) was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

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.

1 participant