[
https://issues.apache.org/jira/browse/STORM-287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13974855#comment-13974855
]
ASF GitHub Bot commented on STORM-287:
--------------------------------------
GitHub user mnogu opened a pull request:
https://github.com/apache/incubator-storm/pull/80
STORM-287: Fix the position of documentation strings
A documentation string should be between a function name
and a parameter list.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mnogu/incubator-storm fix-doc-string-pos
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-storm/pull/80.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #80
----
commit 3569e2265cebb4c6f48291c5cb57163aba98e4da
Author: Muneyuki Noguchi <[email protected]>
Date: 2014-04-19T12:56:54Z
Fix the position of documentation strings.
A documentation string should be between a function name
and a parameter list.
----
> Fix the position of documentation strings
> -----------------------------------------
>
> Key: STORM-287
> URL: https://issues.apache.org/jira/browse/STORM-287
> Project: Apache Storm (Incubating)
> Issue Type: Bug
> Reporter: Muneyuki Noguchi
> Priority: Minor
>
> A documentation string should be between a function name
> and a parameter list, but some strings are not.
> {noformat}
> (defn number-duplicates [coll]
> "(number-duplicates [\"a\", \"b\", \"a\"]) => [\"a\", \"b\", \"a#2\"]"
> (map-occurrences (fn [x occurences] (if (>= occurences 2) (str x "#"
> occurences) x)) coll))
> {noformat}
> {noformat}
> (defn metrics-consumer-register-ids [storm-conf]
> "Generates a list of component ids for each metrics consumer
> e.g. [\"__metrics_org.mycompany.MyMetricsConsumer\", ..] "
> (->> (get storm-conf TOPOLOGY-METRICS-CONSUMER-REGISTER)
> (map #(get % "class"))
> (number-duplicates)
> (map #(str Constants/METRICS_COMPONENT_ID_PREFIX %))))
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.2#6252)
