Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
When issuing a query for IDs that exceed the requested page size, solr returns the full list of results rather than the requested page size. This causes the catalog framework to throw away those results beyond the page size limit and give the impression to any calling code that the next page could be requested. However, since ID queries are changed to solr real time get requests, requesting the next page doesn't work. Solr ignores all of the paging parameters when utilizing the real time get handler. This means that solr will again return the full list of documents and cause the catalog framework to return the same list of results it did for page 1 as page 2. This PR works around this limitation such that if paging parameters are received it will not do a real time get request.
Who is reviewing it?
@pklinef
@clockard
Select relevant component teams:
@codice/solr
Ask 2 committers to review/merge the PR and tag them here.
@glenhein
@jlcsmith
How should this be tested?
Any background context you want to provide?
What are the relevant tickets?
Fixes: #____
Screenshots
Checklist:
Notes on Review Process
Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.
Review Comment Legend: