On Mon, May 23, 2011 at 11:45 AM, Dan Bode <[email protected]> wrote:

> - this ensures that resources are unique based on
>  composite namevars.
> - I need to review why this is required..
>
> Signed-off-by: Dan Bode <[email protected]>
> ---
> Local-branch: issue/master/7629
>  lib/puppet/transaction.rb |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/lib/puppet/transaction.rb b/lib/puppet/transaction.rb
> index 3728a2f..9e40d44 100644
> --- a/lib/puppet/transaction.rb
> +++ b/lib/puppet/transaction.rb
> @@ -226,7 +226,9 @@ class Puppet::Transaction
>     @catalog.vertices.each do |resource|
>       if provider = resource.provider and
> provider.class.respond_to?(:prefetch)
>         prefetchers[provider.class] ||= {}
> -        prefetchers[provider.class][resource.name] = resource
> +        unique_key = resource.uniqueness_key
> +        unique_key = unique_key.to_s if unique_key.size == 1
> +        prefetchers[provider.class][unique_key] = resource
>

MQR - maybe title should be used for uniqueness here?


>       end
>     end
>
> --
> 1.6.5.1
>
> --
> You received this message because you are subscribed to the Google Groups
> "Puppet Developers" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected].
> For more options, visit this group at
> http://groups.google.com/group/puppet-dev?hl=en.
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to