These are chat archives for ManageIQ/manageiq/performance

15th
Mar 2018
Keenan Brock
@kbrock
Mar 15 2018 14:15
@/all so... watched a foreman demo and they showed off https://github.com/wvanbergen/scoped_search
it looks very powerful and has similar concepts to our MiqExpression
it is a pure sql kind of thing. so you can't run the expression on a ruby set. nor can you have ruby only operators
it also does not know about our relations (I think) so it can't do stuff like hosts that have more than 100 vms (though, I take the challenge from @gtanzillo that I may be able to add it ;) )
also wonder if we can add ruby implementation to the filtering
I just wonder if we want to do a gap analasys of the 2 technologies
TOPIC 2: foreman is curious about going from their taxonomy more towards a tagging system. It is probably mostly a u/x issue for them, but they are asking.
I wonder if we can get our tags more hierarchy focused (ancestry?) and possibly simplify classification/category/tagging/tag/ar_taggable
really looking forward to cross pollinating ideas between these 2 rails projects.
Jason Frey
@Fryguy
Mar 15 2018 18:06
for our tagging implementation, I really don't understand why we have the classifications table at all (which while named that, really just holds categories)
we have an entry in the tags table for every tag, but also every category
then every tag has a parent_id
but ones that don't have a parent_id have a corresponding entry in the classifications table
so, there's this mess of repeatability
I;ve thought for a while we can just consolidate the two tables, and use regular ancestry in there for muti-level tags
Keenan Brock
@kbrock
Mar 15 2018 18:27
@Fryguy ok. thanks. that makes sense.
Jason Frey
@Fryguy
Mar 15 2018 18:28
*mess of repetition, I meant to say
Keenan Brock
@kbrock
Mar 15 2018 18:28
I got what you meant
Jason Frey
@Fryguy
Mar 15 2018 18:28
:)
Keenan Brock
@kbrock
Mar 15 2018 18:28
it looked redundant to me
so hearing you say "it is redundant" lets me know that I actually understand it a little - and I'm not just being obtuse
Jason Frey
@Fryguy
Mar 15 2018 18:29
frankly, we could just improve that directly in manageiq and likely get a decent performance benefit
yup
Keenan Brock
@kbrock
Mar 15 2018 18:29
can haz more ancestry please?
Jason Frey
@Fryguy
Mar 15 2018 18:29
:)
Keenan Brock
@kbrock
Mar 15 2018 18:30
I totally want
class Model1
  has_ancestry :column => 'parents', :schema => 'closure_tree'
end
but I don't have good performance tests in there
Libor Pichler
@lpichler
Mar 15 2018 18:37
Maybe redundancy here helped to solve some perfomance issues. NOTE: We are also using tagging/classifications mechanism in assigment mixin, for example for assigning chargeback rates, so it is used not only for tagging.
Keenan Brock
@kbrock
Mar 15 2018 21:20
@lpichler thnx