-
Notifications
You must be signed in to change notification settings - Fork 0
breaking up simulator code #69
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
breaking up simulator code #69
Conversation
src/tasks/sim/factory.rs
Outdated
Db: DatabaseRef + Clone + Sync, | ||
Insp: Sync, | ||
{ | ||
type Database = CacheOnWrite<Db>; |
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.
Why the change from Arc<ConcurrentState> to only CacheOnWrite?
} | ||
|
||
impl<In, Out, S> SimOutcome<In, Out, S> { | ||
/// Creates a new `Best` instance. |
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.
nit: update comments which still call this Best
for handle in handles { | ||
let result = handle.join().unwrap(); | ||
if let Some(candidate) = result { | ||
if *candidate.score() > best_score { |
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.
note: if the scores for all transactions are zero then best
will remain None
(this might be what we want the behavior to be, or we might prefer that it returns one of the transactions?)
closing in favor of sdk work |
based on #63