Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix for issue 212 - Failure to create cluster heatmap #222

Merged
merged 3 commits into from

4 participants

@pcpiela

I was able to reproduce the issue by simulating a cluster with a number of hosts that does not have an exact integer square root, and tested the fix in the same way. Sorry for any incovenience.

Peter

pcpiela added some commits
pcpiela Merge remote branch 'upstream/master' 2bf4c95
pcpiela Merge remote branch 'upstream/master' 3af3bc5
pcpiela Fix for issue 212. The load distribution heatmap would fail to be cre…
…ated for clusters with a number of hosts that did not have an exact integer square root.
f5dce52
@vvuksan vvuksan merged commit 3503296 into ganglia:master
@vvuksan
Owner
@aborn

good job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 17, 2013
  1. Merge remote branch 'upstream/master'

    pcpiela authored
Commits on Jan 4, 2014
  1. Merge remote branch 'upstream/master'

    pcpiela authored
  2. Fix for issue 212. The load distribution heatmap would fail to be cre…

    pcpiela authored
    …ated for clusters with a number of hosts that did not have an exact integer square root.
This page is out of date. Refresh to see the latest.
Showing with 5 additions and 1 deletion.
  1. +5 −1 cluster_view.php
View
6 cluster_view.php
@@ -546,8 +546,12 @@ function get_load_heatmap($hosts_up, $host_regex, $metrics, $data) {
$col_index++;
}
- if ($col_index != 0)
+ if ($col_index != 0) {
+ for ($i = 0; $i < ($num_cols * $num_cols - $num_hosts); $i++) {
+ $heatmap .= ",{host:\"unused\",load:0}";
+ }
$heatmap .= ']';
+ }
$heatmap .= ']';
$data->assign("heatmap_data", $heatmap);
Something went wrong with that request. Please try again.