Skip to content

permit all allowed step sizes when computing#1573

Merged
brharrington merged 2 commits intoNetflix:mainfrom
brharrington:step
Aug 3, 2023
Merged

permit all allowed step sizes when computing#1573
brharrington merged 2 commits intoNetflix:mainfrom
brharrington:step

Conversation

@brharrington
Copy link
Copy Markdown
Contributor

Before there was a separate set of auto step sizes that was a subset of the allowed step sizes. This can sometimes mean the jump is larger than some users would like. The original reason for this was to ensure a higher hit rate for an old caching setup, but is no longer relevant.

Before there was a separate set of auto step sizes that was
a subset of the allowed step sizes. This can sometimes mean
the jump is larger than some users would like. The original
reason for this was to ensure a higher hit rate for an old
caching setup, but is no longer relevant.
@brharrington brharrington added this to the 1.8.0 milestone Aug 3, 2023
@brharrington brharrington requested a review from manolama August 3, 2023 20:55
@brharrington brharrington merged commit f2d7cc4 into Netflix:main Aug 3, 2023
@brharrington brharrington deleted the step branch August 3, 2023 21:37
manolama pushed a commit to manolama/atlas that referenced this pull request May 22, 2024
Before there was a separate set of auto step sizes that was
a subset of the allowed step sizes. This can sometimes mean
the jump is larger than some users would like. The original
reason for this was to ensure a higher hit rate for an old
caching setup, but is no longer relevant.
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.

2 participants