These are chat archives for ManageIQ/manageiq/performance

6th
Sep 2016
Keenan Brock
@kbrock
Sep 06 2016 12:59

So Crystal is a ruby like language.
The thought is to use crystal for native extensions over C.
less impedence mismatch.

Interesting to see a "native" ActiveSupport::Inflector.
https://github.com/phoffer/fast_inflector

/via https://github.com/phoffer/crystalized_ruby

I have wanted to add some more for other screens, but haven't gotten around to it
But services is pretty easy
Instructions in the README if you want to just use the gem
Keenan Brock
@kbrock
Sep 06 2016 15:46
thanks so much
@Fryguy thanks so much for merging ManageIQ/manageiq#10961
Keenan Brock
@kbrock
Sep 06 2016 15:52
@NickLaMuro / @dmetzger57 I'm going to split ManageIQ/manageiq#11003 into 2 prs and add screen shots
Jason Frey
@Fryguy
Sep 06 2016 15:53
@kbrock It's great stuff @kbrock
Keenan Brock
@kbrock
Sep 06 2016 19:44

@Fryguy Is there a performance reason for staying with get_vmdb_config? I'm guessing it probably caches the hash a little better.

  def get_vmdb_config
    @vmdb_config ||= VMDB::Config.new("vmdb").config
  end
# get_vmdb_config[:server][:value]

vs replacing all the callers (1) with:

Settings.server.value
for the tree builder stuff, there is only 1 caller, and it is read only - so would be an easy change for Settings - but unsure to the speed hit implications
Jason Frey
@Fryguy
Sep 06 2016 20:04
No, we should be able to replace
Actually, Settings call should use less memory
I've been doing them case by case