Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changes after inventory refresh memory optimizations #109

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented Jan 19, 2017

Will be fixing a CI after ManageIQ/manageiq#13565 is merged

This patch brings needed ext_management_system relation to each parsed item and optimizes how we store Relationships and Ancestry

Ladas added 3 commits January 19, 2017 10:34
Associate ems to each relevant enitity in the parser
For performance reasons, we will be solving Ancestry and
Relationship attributes as a separate nodes in a graph.
Custom nodes and saving for Relationship and Ancestry, getting
all records involved in one query will solve us thousands of
queries to the DB.
@Ladas Ladas closed this Jan 19, 2017
@Ladas Ladas reopened this Jan 19, 2017
Storage managers need to have ems associated
@miq-bot
Copy link
Member

miq-bot commented Jan 19, 2017

Checked commits Ladas/manageiq-providers-amazon@ef545fe~...a970267 with ruby 2.2.6, rubocop 0.46.0, and haml-lint 0.19.0
6 files checked, 18 offenses detected

app/models/manageiq/providers/amazon/cloud_manager/refresh_parser_inventory_object.rb

app/models/manageiq/providers/amazon/inventory/targets/cloud_manager.rb

app/models/manageiq/providers/amazon/network_manager/refresh_parser_inventory_object.rb

@durandom durandom merged commit 590afc5 into ManageIQ:master Jan 20, 2017
@bronaghs bronaghs mentioned this pull request Jan 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants