ManageIQ / inventory_refresh

Apache License 2.0
1 stars 23 forks source link

Remove inefficient saving and deleting strategies #70

Closed Ladas closed 5 years ago

Ladas commented 5 years ago

Only :concurrent_safe_batch saver strategy and effective retention_strategies (:archive and :destroy) are allowed, the rest is deleted.

Depends on:

miq-bot commented 5 years ago

This pull request is not mergeable. Please rebase and repush.

miq-bot commented 5 years ago

Checked commits https://github.com/Ladas/inventory_refresh/compare/3241f663d456ba5c5618548f876f99b05323a0d6~...5093eedeb93ad69e75403450bea1f9c3c3a3352c with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 38 files checked, 44 offenses detected

lib/inventory_refresh/inventory_collection/builder.rb

spec/persister/local_db_finders_spec.rb

spec/save_inventory/acyclic_graph_of_inventory_collections_spec.rb

spec/save_inventory/graph_of_inventory_collections_spec.rb

spec/schema.rb