Improve cluster load-heatmap implementation #212

Merged
merged 2 commits into from Dec 31, 2013

Projects

None yet

3 participants

@pcpiela
Contributor
pcpiela commented Oct 27, 2013

Create a Javascript class for the heatmap
Combine load and host name data into a single object array

@vvuksan vvuksan merged commit d736b75 into ganglia:master Dec 31, 2013
@analytically

This fails for me in Firefox 26 and Chrome 31.
screenshot from 2014-01-04 21 33 36

@pcpiela
Contributor
pcpiela commented Jan 4, 2014

I will try to reproduce, and make a fix.

Peter

From: Mathias Bogaert [mailto:notifications@github.com]
Sent: Saturday, January 04, 2014 4:35 PM
To: ganglia/ganglia-web
Cc: Peter Piela
Subject: Re: [ganglia-web] Improve cluster load-heatmap implementation (#212)

This fails for me in Firefox 26 and Chrome 31.
[screenshot from 2014-01-04 21 33 36]https://f.cloud.github.com/assets/254434/1845413/06113e4a-7588-11e3-9206-0798a4190316.png


Reply to this email directly or view it on GitHubhttps://github.com/ganglia/ganglia-web/pull/212#issuecomment-31589223.

@analytically

Can you reproduce? With the same setup it was working fine using Ganglia Web 3.5.10. I'm also seeying this in my Hadoop Ansible playbook: https://github.com/analytically/hadoop-ansible (check out the build screenshots)

@pcpiela
Contributor
pcpiela commented Jan 6, 2014

I have reproduced the issue and submitted a pull request for the fix, cluster_view.php

Let me know if you have problems picking it up. Sorry for any inconvenience.

Peter

Sent from my Android phone using TouchDown (www.nitrodesk.com)

-----Original Message-----
From: Mathias Bogaert [notifications@github.com]
Received: Monday, 06 Jan 2014, 6:46AM
To: ganglia/ganglia-web [ganglia-web@noreply.github.com]
CC: Peter Piela [Peter.Piela@terascala.com]
Subject: Re: [ganglia-web] Improve cluster load-heatmap implementation (#212)

Can you reproduce? With the same setup it was working fine using Ganglia Web 3.5.10. I'm also seeying this in my Hadoop Ansible playbook: https://github.com/analytically/hadoop-ansible (check out the build screenshots)

Reply to this email directly or view it on GitHubhttps://github.com/ganglia/ganglia-web/pull/212#issuecomment-31643103.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment