Skip to content

[SPARK-8927] [DOCS] Format wrong for some config descriptions #7292

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.

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

Closed
wants to merge 1 commit into from

Conversation

jonalter
Copy link
Contributor

@jonalter jonalter commented Jul 8, 2015

A couple descriptions were not inside <td></td> and were being displayed immediately under the section title instead of in their row.

@srowen
Copy link
Member

srowen commented Jul 8, 2015

LGTM

@sarutak
Copy link
Member

sarutak commented Jul 8, 2015

ok to test.

@SparkQA
Copy link

SparkQA commented Jul 8, 2015

Test build #36834 has finished for PR 7292 at commit 5ce1570.

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

@sarutak
Copy link
Member

sarutak commented Jul 8, 2015

@jonalter can you file this in JIRA and add the issue ID to this title?

@jonalter jonalter changed the title [DOCS] Format wrong for some config descriptions [SPARK-8927] [DOCS] Format wrong for some config descriptions Jul 8, 2015
asfgit pushed a commit that referenced this pull request Jul 9, 2015
A couple descriptions were not inside `<td></td>` and were being displayed immediately under the section title instead of in their row.

Author: Jonathan Alter <[email protected]>

Closes #7292 from jonalter/docs-config and squashes the following commits:

5ce1570 [Jonathan Alter] [DOCS] Format wrong for some config descriptions

(cherry picked from commit 28fa01e)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in 28fa01e Jul 9, 2015
@jonalter jonalter deleted the docs-config branch July 9, 2015 21:18
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.

4 participants