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.

Sign up for GitHub

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

Jump to bottom

Fixes #6059: Graphes in home page take ages to display #755

Merged
merged 1 commit into from
Dec 31, 2014
Merged

Fixes #6059: Graphes in home page take ages to display #755

merged 1 commit into from
Dec 31, 2014

Conversation

fanf
Copy link
Member

@fanf fanf commented Dec 31, 2014

There is three main item here:

  • the addition of a logger for Timing-related info,
  • the split of the big ajax call for all graphes in 3 (one for each kind of data needed in the background: node infos, software, compliance)
  • massive optimisation of software retrieval (from O(nb_node) to O(1)) and optimisation of nodeInfo (~ time/2)

ncharles
ncharles reviewed Dec 31, 2014
View reviewed changes
rudder-core/src/main/scala/com/normation/rudder/services/nodes/NodeInfoService.scala
} else if(e.isA(OC_RUDDER_NODE)) {
nodes += (e.value_!(A_NODE_UUID) -> e)
} else {
// it's an error, don't use
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will you have also the policy server with this ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

@ncharles
Copy link
Member

This is a welcomed change, thank you

ncharles added a commit that referenced this pull request Dec 31, 2014
@ncharles
Merge pull request #755 from fanf/bug_6059/graphes_in_home_page_take_…
d06f050 
…ages_to_display

Fixes #6059: Graphes in home page take ages to display
@ncharles ncharles merged commit d06f050 into Normation:branches/rudder/3.0 Dec 31, 2014
@fanf fanf deleted the bug_6059/graphes_in_home_page_take_ages_to_display branch March 15, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@fanf @ncharles