-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 02/07/2014 08:26 PM, John Bollinger wrote: > > > On Friday, February 7, 2014 3:57:00 AM UTC-6, Brice Figureau > wrote: > > In this specific bug, I'm wondering if we really need to have > those defined type as resources in the RAL catalog (I believe it > might be used for relationship or event propagation). > > > > I suspect that defined type instances could be removed from the > catalog by replacing them with the collections of resources the > represent (recursively). However, I would expect that to cause an > explosion in the number of relationships (both signaling and > non-signaling) that must appear in the catalog.
"Flattening" the catalog is something that we have discussed before [1] and which would give imho quite some improvements. However, we would likely loose important context information, that should then be preserved differently. Coming back to the "Bug": I think swapping the statement would give already some improvement and is a fix that could go into the next (bugfix)-release. Given that there are no concerns regarding possible side effects. Still it shows that there is room for improvement in general within this area that should be part of a bigger review, re-architecture. Nevertheless, I think we should also make the tiny steps that don't hurt but give some improvement. ~pete [1] https://groups.google.com/forum/#!topic/puppet-dev/BPvuDlb67WE -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iEYEARECAAYFAlL1bjwACgkQbwltcAfKi3/WRgCfUi/M9D1pebXlxs/QImwkXOV3 h9MAn1TEGLKJJvdezC1ZGZZNTxHEpVgY =IAtQ -----END PGP SIGNATURE----- -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/puppet-dev/52F56E3D.8030103%40immerda.ch. For more options, visit https://groups.google.com/groups/opt_out.
