-
Notifications
You must be signed in to change notification settings - Fork 492
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
Add resources
argument in EvalForIndexer()
#2952
Add resources
argument in EvalForIndexer()
#2952
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.
the change looks fine - but I'd like to request adding some testing for the saveResourcesInCowBase
method. Testing it individually or via the EvalForIndexer
wrapper is up to you.
i.e. the goal is to prove that the roundCowBase here would truly initialize its state by the method. The standard cowRoundBase interface should be used to validate the correctness rather than direct access to the underlying maps.
Codecov Report
@@ Coverage Diff @@
## master #2952 +/- ##
=======================================
Coverage 47.61% 47.61%
=======================================
Files 358 358
Lines 57906 57889 -17
=======================================
- Hits 27570 27563 -7
+ Misses 27208 27201 -7
+ Partials 3128 3125 -3
Continue to review full report at Codecov.
|
51255fc
to
a95f0fa
Compare
I added a test. Is this what you meant? |
merged in #3019 |
Summary
This way Indexer can preload most things from the database efficiently.