Skip to content

Conversation

@harishreedharan
Copy link
Contributor

Minor error in the monitoring docs. Also made indentation changes in ApiRootResource

@andrewor14
Copy link
Contributor

Thanks for doing this after merge @harishreedharan. The changes here lgtm.

@harishreedharan
Copy link
Contributor Author

I am adding the docs you mentioned in the other PR. Please hold off for a couple mins before merging.

@harishreedharan
Copy link
Contributor Author

@andrewor14, that should be it.

@andrewor14
Copy link
Contributor

merging into master since the changes here are just style and docs.

@asfgit asfgit closed this in 566cb59 Jun 3, 2015
@SparkQA
Copy link

SparkQA commented Jun 4, 2015

Test build #34134 has finished for PR 6628 at commit ca399b6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 4, 2015

Test build #34135 timed out for PR 6628 at commit a12553d after a configured wait of 150m.

@harishreedharan harishreedharan deleted the eventlog-formatting branch June 9, 2015 05:45
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
Minor error in the monitoring docs. Also made indentation changes in `ApiRootResource`

Author: Hari Shreedharan <[email protected]>

Closes apache#6628 from harishreedharan/eventlog-formatting and squashes the following commits:

a12553d [Hari Shreedharan] Javadoc updates.
ca399b6 [Hari Shreedharan] [HOTFIX] History Server API docs error fix.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
Minor error in the monitoring docs. Also made indentation changes in `ApiRootResource`

Author: Hari Shreedharan <[email protected]>

Closes apache#6628 from harishreedharan/eventlog-formatting and squashes the following commits:

a12553d [Hari Shreedharan] Javadoc updates.
ca399b6 [Hari Shreedharan] [HOTFIX] History Server API docs error fix.
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