-
Notifications
You must be signed in to change notification settings - Fork 46
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
feat(core)!: search optional count and return only SearchResult #1200
feat(core)!: search optional count and return only SearchResult #1200
Conversation
Test Results 4 files ±0 4 suites ±0 5m 56s ⏱️ +2s Results for commit 9347fcc. ± Comparison against base commit e577b27. This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Code Coverage (Ubuntu)
Diff against develop
Results for commit: 9347fcc Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Code Coverage (Windows)
Diff against develop
Results for commit: 9347fcc Minimum allowed coverage is ♻️ This comment has been updated with latest results |
6b94645
to
ff2bba3
Compare
dce6825
to
4cfc99b
Compare
e5ade76
to
3393bb7
Compare
Add new property
SearchResult.estimated_total_results
. The return value of the following functions has been modified accordingly to the new property, returning only aSearchResult
object:EODataAccessGateway.search
EODataAccessGateway._search_by_id
EODataAccessGateway._do_search
Add a new
count
parameter toEODataAccessGateway.search
to disable count requests.