diff options
author | Brice Figureau <brice@daysofwonder.com> | 2008-08-11 10:52:50 +0200 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2008-08-17 23:47:30 +1000 |
commit | 2ec4e298c3274abc8eaad4230bca8d39a48d2e35 (patch) | |
tree | 6079621408d5d2291ac77648b3f5ef47564ad2dd /test/ral | |
parent | 9272df437c3cee6070f4e765bf61511cd85cb3a7 (diff) | |
download | puppet-2ec4e298c3274abc8eaad4230bca8d39a48d2e35.tar.gz puppet-2ec4e298c3274abc8eaad4230bca8d39a48d2e35.tar.xz puppet-2ec4e298c3274abc8eaad4230bca8d39a48d2e35.zip |
Fix #1502 - abysmal storeconfig performance - part2
Resource parameters whose values are a resource reference (ie require,
notify...) where always DELETEd/INSERTed because the code comparing
resource reference compared object instances instead of their values
(since Puppet::Parser::Resource::Reference doesn't override == ), leading
to storeconfig performance issues.
The correct fix would have been to define == in Puppet::Parser::Resource::Reference
but that might introduce some side effects I don't know.
Hence, the fix introduces a local compare() method that knows how to
compare Puppet::Parser::Resource::Reference.
Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
Diffstat (limited to 'test/ral')
0 files changed, 0 insertions, 0 deletions