Skip to content

Conversation

@Daemoen
Copy link
Contributor

@Daemoen Daemoen commented Aug 15, 2013

Ops teams need to ensure that the cluster is functional and performant. Having to scrape the html source for worker state won't work reliably, and will be slow. By exposing the state in the json output, ops teams are able to ensure a fully functional environment by querying for the json output and parsing for dead nodes.

Ops teams need to ensure that the cluster is functional and performant.  Having to scrape the html source for worker state won't work reliably, and will be slow.  By exposing the state in the json output, ops teams are able to ensure a fully functional environment by querying for the json output and parsing for dead nodes.
@AmplabJenkins
Copy link

Thank you for your pull request. An admin will review this request soon.

@rxin
Copy link
Member

rxin commented Aug 16, 2013

Jenkins, test this please.

rxin added a commit that referenced this pull request Aug 16, 2013
Updated json output to allow for display of worker state
@rxin rxin merged commit 230ab27 into mesos:master Aug 16, 2013
rxin added a commit that referenced this pull request Aug 16, 2013
This reverts commit 230ab27, reversing
changes made to 659553b.
@rxin
Copy link
Member

rxin commented Aug 16, 2013

Actually I accidentally merged this one. This pull request as is won't build, because WorkerInfo doesn't have the getState method. I am working on something related to this file. I will fix it based on that.

@rxin
Copy link
Member

rxin commented Aug 16, 2013

See #841

@AmplabJenkins
Copy link

Thank you for submitting this pull request.

Unfortunately, the automated tests for this request have failed.

Refer to this link for build results: http://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/611/

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