[ 
https://issues.apache.org/jira/browse/STORM-3931?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bipin Prasad reassigned STORM-3931:
-----------------------------------

    Assignee: Bipin Prasad

> @see tags that are followed by {@link...}
> -----------------------------------------
>
>                 Key: STORM-3931
>                 URL: https://issues.apache.org/jira/browse/STORM-3931
>             Project: Apache Storm
>          Issue Type: Task
>          Components: documentation
>            Reporter: Bipin Prasad
>            Assignee: Bipin Prasad
>            Priority: Major
>
> @see tags have been deprected in JDK9 in favor of @link tags.
> In Storm, several source files contain improperly formatted '@see' tags. This 
> tag expects class name otherwise generates error for unexpected "{" - 
> character code 123 and "@", character code 64. When generating javadoc.
> {code:java}
> [INFO] [WARNING] 
> .../storm/storm-client/src/jvm/org/apache/storm/utils/DefaultShellLogHandler.java:51:
>  warning - Tag @see:illegal character: "123" in "{@link 
> ShellLogHandler#setUpContext}"
> [INFO] [WARNING] 
> .../storm/storm-client/src/jvm/org/apache/storm/utils/DefaultShellLogHandler.java:51:
>  warning - Tag @see:illegal character: "64" in "{@link 
> ShellLogHandler#setUpContext}"
> [INFO] [WARNING] 
> .../storm/storm-client/src/jvm/org/apache/storm/utils/DefaultShellLogHandler.java:51:
>  warning - Tag @see: reference not found: "{@link 
> ShellLogHandler#setUpContext}"
> {code}
> For example
> storm-client/src/jvm/org/apache/storm/utils/DefaultShellLogHandler.java#setUpContext
> {code:java}
> /**
>  * This default implementation saves the {@link ShellProcess} so it can 
> output the process info string later.
>  *
>  * @param ownerCls - the class which instantiated this ShellLogHandler.
>  * @param process  - the current {@link ShellProcess}.
>  * @param context  - the current {@link TopologyContext}.
>  * @see {@link ShellLogHandler#setUpContext}
>  */
>  {code}
> Should just be rolled into the documentation thusly:
> {code:java}
> /**
>  * This default implementation saves the {@link ShellProcess} so it can 
> output the process info string later. See {@link 
> ShellLogHandler#setUpContext}.
>  *
>  * @param ownerCls - the class which instantiated this ShellLogHandler.
>  * @param process  - the current {@link ShellProcess}.
>  * @param context  - the current {@link TopologyContext}.
>  */
>  {code}
> or
> {code:java}
> /**
>  * This default implementation saves the {@link ShellProcess} so it can 
> output the process info string later.
>  *
>  * @param ownerCls - the class which instantiated this ShellLogHandler.
>  * @param process  - the current {@link ShellProcess}.
>  * @param context  - the current {@link TopologyContext}.
>  * @see ShellLogHandler#setUpContext
>  */
>  {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to