These are chat archives for ManageIQ/manageiq/performance

2nd
Mar 2018
Keenan Brock
@kbrock
Mar 02 2018 18:59
yea - set is much quicker. right?
Daniel Berger
@djberg96
Mar 02 2018 19:00
alot, yes
Ron Valente
@rvalente
Mar 02 2018 19:00
:hearts: Sets
Keenan Brock
@kbrock
Mar 02 2018 19:00
Set has a Hash inside of it
Daniel Berger
@djberg96
Mar 02 2018 19:00
was just looking at it for targeted refresh where we're checking for individual ems_ref's
Keenan Brock
@kbrock
Mar 02 2018 19:01
we used to convert to sets all over the place before
but so much code
Daniel Berger
@djberg96
Mar 02 2018 19:01
but not now?
Ron Valente
@rvalente
Mar 02 2018 19:01
Sets also allow no duplicates
Daniel Berger
@djberg96
Mar 02 2018 19:03
wouldn't expect any in a refresh
even if it showed up more than once, just refresh!
looks like the constructor just compacts dups