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
Fix index count on IndexSetPage #11395
Fix index count on IndexSetPage #11395
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Looks good and works. Can you run yarn lint:changes --fix
and commit the resulting diff? The spacing is off in your change.
Oops, sorry about that. I've ran |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot, @supahgreg! Good work!
* Fix index count on IndexSetPage. * Fix spacing in IndexSetPage. (cherry picked from commit 01253de)
@supahgreg: If you are interested in a little thank you from our side, feel free to drop me your details in a short email! |
* Fix index count on IndexSetPage. * Fix spacing in IndexSetPage.
Fixes the index count on
IndexSetPage
.Description
Previously,
_totalIndexCount
was counting the keys of theindexerOverview
response (introduced w/ #10401; specifically this), rather than the response'sindices
key. This PR updates that function so it counts the right thing.Motivation and Context
Fixes #11375.
How Has This Been Tested?
It hasn't been tested.
Types of changes
Checklist: