Skip to content

Conversation

@kennyballou
Copy link
Contributor

Fix several awkward wordings and grammatical issues in the following
documents:

  • docs/monitoring.md
  • docs/streaming-programming-guide.md

Fix several awkward wordings and grammatical issues in the following
documents:

*   docs/monitoring.md

*   docs/streaming-programming-guide.md
@srowen
Copy link
Member

srowen commented Jul 30, 2014

Meta question: for trivial typo fixes, comment fixes, or non-functional code changes, is a JIRA needed? I'd personally say no.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@JoshRosen
Copy link
Contributor

@srowen We don't need a JIRA for patches that only fix small documentation typos, since we won't need a ticket to refer to in later discussions.

@JoshRosen
Copy link
Contributor

I've merged this into master and branch-1.0. Thanks!

@asfgit asfgit closed this in cc82050 Jul 31, 2014
asfgit pushed a commit that referenced this pull request Jul 31, 2014
Fix several awkward wordings and grammatical issues in the following
documents:

*   docs/monitoring.md

*   docs/streaming-programming-guide.md

Author: kballou <[email protected]>

Closes #1662 from kennyballou/grammar_fixes and squashes the following commits:

e1b8ad6 [kballou] Docs: monitoring, streaming programming guide

(cherry picked from commit cc82050)
Signed-off-by: Josh Rosen <[email protected]>
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Fix several awkward wordings and grammatical issues in the following
documents:

*   docs/monitoring.md

*   docs/streaming-programming-guide.md

Author: kballou <[email protected]>

Closes apache#1662 from kennyballou/grammar_fixes and squashes the following commits:

e1b8ad6 [kballou] Docs: monitoring, streaming programming guide
@kennyballou kennyballou deleted the grammar_fixes branch April 22, 2015 17:09
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